-
Notifications
You must be signed in to change notification settings - Fork 424
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
Added crate_universe examples to CI #707
Conversation
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.
This looks great, surprisingly nice and clean! Thanks for putting so much effort into it, it definitely paid off!
Can you also remove the now-stale bootstrap bazelrc files from the .gitignores? When testing this out locally they tripped me up, and if they'd shown up in my git status
it would've saved me some head-scratching :)
Co-authored-by: Daniel Wagner-Hall <dawagner@gmail.com>
@illicitonion ready for another pass 😄 |
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.
Looks great - thanks!
This change introduces a new
crate_universe_bootstrap
macro which defines a repository rule for bootstrappingcrate_universe_resolver
binaries.crate_universe
can now be made to bootstrap a binary (instead of downloading one) by using theRULES_RUST_CRATE_UNIVERSE_BOOTSTRAP
environment variable and adding thecrate_universe_bootstrap
macro to yourWORKSPACE
file.The goal of this PR is to enable testing for
crate_universe
which was originally solved by the./bootstrap.sh
script but this unfortunately lead to many issues. Using a repository rule to compose all the work needed for bootstrapping basically solved for all of these.