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

[8.0.0] Backport MODULE.bazel file changes #24481

Merged

Conversation

meteorcloudy
Copy link
Member

- Embedded some test dependencies in MODULE.bazel with use_repo_rule, this avoids loading some .bzl files under src/test when building Bazel (and reduced some boilerplate).
- Use pkg_files from rules_pkg to exclude some sources that are not needed in the bazel distfile (e.g test sources and documentation).

With this change, touching only tests and documentation code should no longer trigger the expensive bootstrap tests.

PiperOrigin-RevId: 698478000
Change-Id: I7fdf49123b6954bab7e5b31795ea657e559bffbe
Addressing bazel-contrib/bazel_features#82

PiperOrigin-RevId: 699970013
Change-Id: Ia9b6952c4836437b4f74e4e498c5d5839185c496
@meteorcloudy meteorcloudy requested a review from a team as a code owner November 25, 2024 15:32
@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Nov 25, 2024
@meteorcloudy meteorcloudy changed the title Backport MODULE.bazel file changes [8.0.0] Backport MODULE.bazel file changes Nov 25, 2024
@meteorcloudy meteorcloudy added this pull request to the merge queue Nov 25, 2024
Merged via the queue into bazelbuild:release-8.0.0 with commit 11596c3 Nov 25, 2024
45 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Nov 25, 2024
@iancha1992 iancha1992 added this to the 8.0.0 release blockers milestone Nov 25, 2024
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.

3 participants