-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[build] Adjust new_release for bzlmod spellings #22482
Conversation
+@BetsyMcPhail for feature review, please (or feel free to delegate). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Output from running bazel run //tools/workspace:new_release
:
master:
crate_universe may need upgrade
rust_toolchain may need upgrade
bazel may need upgrade but cannot be auto-upgraded.
python may need upgrade but cannot be auto-upgraded.
This branch:
abseil_cpp_internal needs upgrade from 60d1ce9911e6b32ea0c5a2fcfc120f9b392d5643 to 655707fb20297711abfddec6c85dcb7c116c8d2e
buildifier needs upgrade from v7.3.1 to v8.0.1
crate_universe may need upgrade
libpng_internal needs upgrade from v1.6.44 to v1.6.45
mujoco_menagerie_internal needs upgrade from 469893211c41d5da9c314f5ab58059fa17c8e360 to 5e6af47a49bf8ae27581615b54045107b20ce584
nanoflann_internal needs upgrade from v1.6.2 to v1.6.3
Skipping prerelease unstable for ros_xacro_internal
ros_xacro_internal needs upgrade from 2.0.11 to 2.0.12
rust_toolchain may need upgrade
spral_internal needs upgrade from v2024.05.08 to v2025.01.08
stable_baselines3_internal needs upgrade from v2.4.0 to v2.4.1
vtk_internal needs upgrade from e43b7215c0f57781626c4cd31c95cd1c1b7a3d48 to 869ef27449bfc976027b63e6b2eedc835661d4e8
bazel may need upgrade but cannot be auto-upgraded.
python may need upgrade but cannot be auto-upgraded.
Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: needs platform reviewer assigned, needs at least two assigned reviewers
+@xuchenhan-tri for platform review (tomorrow), please. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: 1 unresolved discussion
tools/workspace/metadata.bzl
line 4 at r1 (raw file):
"""Helper function to archive load-phase metadata. Writes a file named drake_repository_metadata.json to the root of an external repository. The contents of the file will a JSON dict of the `kwargs`, with one additional
nit,
Suggestion:
drake_repository_metadata.json to the root of an external repository. The
contents of the file will be a JSON dict of the `kwargs`, with one additional
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: complete! all discussions resolved, LGTM from assignees xuchenhan-tri(platform),BetsyMcPhail
Towards #20731.
This change is