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

chore: add //bzlmod:hello_world_test and use it as BCR presubmit #2347

Merged
merged 23 commits into from
Dec 27, 2023

Conversation

cgrindel
Copy link
Contributor

@cgrindel cgrindel commented Dec 20, 2023

  • Create bzlmod/hello_world package based upon examples/bzlmod/hello_world.
  • Add rules_bazel_integration_test so that //bzlmod:hello_world_test can be executed as a test from the parent workspace.
  • Add filegroup targets named all_files to collect files for all of the packages that are necessary to use rules_rust. These are an input to the rules_bazel_integration_test test.
  • Replace .bcr/presubmit.yml to test //bzlmod:hello_world_test.
  • Update CI to test //bzlmod:hello_world_test.

Related to bazelbuild/bazel-central-registry#1199 (comment).

@cgrindel cgrindel marked this pull request as draft December 20, 2023 21:30
@cgrindel cgrindel marked this pull request as ready for review December 20, 2023 23:35
.gitignore Outdated Show resolved Hide resolved
.bazelrc Outdated Show resolved Hide resolved
MODULE.bazel Outdated Show resolved Hide resolved
MODULE.bazel Outdated Show resolved Hide resolved
.bazelrc Outdated Show resolved Hide resolved
@cgrindel
Copy link
Contributor Author

@matts1 @illicitonion I removed rules_bazel_integration_test from the PR. This should be ready for another review.

BTW, the GitHub UI is showing the .bcr/presubmit.yml file as being deleted. It should not be deleted. The branch shows that it is there. If it appears deleted for you, please let me know.

Copy link
Collaborator

@illicitonion illicitonion left a 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/BUILD.bazel Show resolved Hide resolved
bzlmod/hello_world/hello_world_test.sh Outdated Show resolved Hide resolved
@illicitonion illicitonion enabled auto-merge (squash) December 27, 2023 16:42
@illicitonion illicitonion merged commit 6a93592 into bazelbuild:main Dec 27, 2023
3 checks passed
@cgrindel cgrindel deleted the cg/bzlmod_presubmit branch December 27, 2023 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants