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

Add script to revendor all crates #2459

Merged
merged 3 commits into from
Feb 18, 2024

Conversation

cameron-martin
Copy link
Contributor

@cameron-martin cameron-martin commented Feb 3, 2024

It seems that vendored crates commonly get out of sync. Re-vendoring seems to currently be a quite manual process, and finding everything to re-vendor is a hassle.

This adds a bash script, vendor.sh, that finds all crate vendor rules in all workspaces in this repository and runs them. This significantly improves the workflow for updating these.

This also adds .bazelrc files to disable bzlmod to non-bzlmod workspaces, since otherwise bazel 7 create a MODULE.bazel and MODULE.bazel.lock when re-vendoring.

This also re-vendors all crates (in a separate commit).

Copy link
Collaborator

@UebelAndre UebelAndre left a comment

Choose a reason for hiding this comment

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

So sorry I didn't see this sooner, This would be fantastic! Just one request

vendor.sh Outdated Show resolved Hide resolved
@cameron-martin cameron-martin force-pushed the revendor-crates branch 2 times, most recently from 8c1c3b8 to eb2a060 Compare February 16, 2024 14:36
It seems that vendored crates commonly get out of sync. Re-vendoring seems to currently be a quite manual process, and finding everything to re-vendor is a hassle.

This adds a bash script, `vendor.sh`, that finds all crate vendor rules in all workspaces in this repository and runs them. This significantly improves the workflow for updating these.
Copy link
Collaborator

@UebelAndre UebelAndre left a comment

Choose a reason for hiding this comment

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

Thanks!

@UebelAndre UebelAndre merged commit 04f1b5d into bazelbuild:main Feb 18, 2024
3 checks passed
@cameron-martin cameron-martin deleted the revendor-crates branch February 18, 2024 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants