-
Notifications
You must be signed in to change notification settings - Fork 446
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
chore: add //bzlmod:hello_world_test
and use it as BCR presubmit
#2347
Conversation
7f0eb2b
to
fcdf36e
Compare
14ac7d8
to
9f8f044
Compare
0ebe128
to
8b8b66b
Compare
@matts1 @illicitonion I removed BTW, the GitHub UI is showing the |
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.
LGTM, thanks!
bzlmod/hello_world
package based uponexamples/bzlmod/hello_world
.rules_bazel_integration_test
so that//bzlmod:hello_world_test
can be executed as a test from the parent workspace.filegroup
targets namedall_files
to collect files for all of the packages that are necessary to userules_rust
. These are an input to therules_bazel_integration_test
test..bcr/presubmit.yml
to test//bzlmod:hello_world_test
.//bzlmod:hello_world_test
.Related to bazelbuild/bazel-central-registry#1199 (comment).