Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[workspace] Withdraw stable API support for several externals #17324

Merged

Conversation

jwnimmer-tri
Copy link
Collaborator

@jwnimmer-tri jwnimmer-tri commented Jun 3, 2022

These are the python- and models-only repositories that should not be public.

The models already have Drake-public filegroup wrappers where appropriate, in the @drake// namespace.

The python libraries are either for maintainer-only code (github3, uritemplate), or else brittle interior dependencies (clang) of Drake-private tools (mkdoc).

Relates to #17231 in the sense of a uniform stable / internal policy, even though there is no C/C++ code here.


This change is Reviewable

@jwnimmer-tri jwnimmer-tri added priority: low release notes: newly deprecated This pull request contains new deprecations labels Jun 3, 2022
@jwnimmer-tri jwnimmer-tri marked this pull request as ready for review June 3, 2022 20:53
@jwnimmer-tri
Copy link
Collaborator Author

+@rpoyner-tri for feature review, please.

@jwnimmer-tri
Copy link
Collaborator Author

@rpoyner-tri what do you think about +(status: single reviewer ok)?

@jwnimmer-tri jwnimmer-tri added the status: single reviewer ok https://drake.mit.edu/reviewable.html label Jun 7, 2022
Copy link
Contributor

@rpoyner-tri rpoyner-tri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

single is fine. Still looking.

Reviewed 50 of 50 files at r1, all commit messages.
Reviewable status: 1 unresolved discussion, LGTM missing from assignee rpoyner-tri(platform) (waiting on @jwnimmer-tri)


-- commits line 7 at r1:
grammar? missing words?

Code quote:

  The python libraries either for maintainer-only code (github3, template),
  or else brittle interior dependencies (clang) of private tools (mkdoc).

Copy link
Contributor

@rpoyner-tri rpoyner-tri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm: new_release.py

Reviewable status: 2 unresolved discussions (waiting on @jwnimmer-tri)

a discussion (no related file):
Similar to a previous PR, are there names in tools/workspace/new_release.py that need updating?


These are the python- and models-only repositories that should not be public.

The models already have Drake-public filegroup wrappers where appropriate.
The python libraries are either maintainer-only code (github3, template),
or else brittle interior dependencies (clang) of private tools (mkdoc).
Copy link
Collaborator Author

@jwnimmer-tri jwnimmer-tri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dismissed @rpoyner-tri from a discussion.
Reviewable status: 1 unresolved discussion (waiting on @rpoyner-tri)

a discussion (no related file):

Previously, rpoyner-tri (Rick Poyner (rico)) wrote…

Similar to a previous PR, are there names in tools/workspace/new_release.py that need updating?

Done.


Copy link
Contributor

@rpoyner-tri rpoyner-tri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: :shipit: complete! all discussions resolved, LGTM from assignee rpoyner-tri(platform) (waiting on @jwnimmer-tri)

@rpoyner-tri rpoyner-tri merged commit 524bbdb into RobotLocomotion:master Jun 7, 2022
@jwnimmer-tri jwnimmer-tri deleted the workspace-internal branch June 7, 2022 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: low release notes: newly deprecated This pull request contains new deprecations status: single reviewer ok https://drake.mit.edu/reviewable.html
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants