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

Don't send unnecessary lockfile event #20154

Closed
wants to merge 1 commit into from
Closed

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented Nov 11, 2023

BazelLockFileModule is the only consumer of BazelModuleResolutionEvent and only subscribes with --lockfile_mode=update.

`BazelLockFileModule` is the only consumer of `BazelModuleResolutionEvent` and only subscribes with `--lockfile_mode=update`.
@fmeum fmeum changed the title Don't send unnecessary lockfile events Don't send unnecessary lockfile event Nov 11, 2023
@fmeum fmeum marked this pull request as ready for review November 11, 2023 22:05
@fmeum fmeum requested review from SalmaSamy and Wyverald and removed request for Wyverald and meteorcloudy November 11, 2023 22:05
@fmeum
Copy link
Collaborator Author

fmeum commented Nov 11, 2023

@bazel-io flag

@github-actions github-actions bot added awaiting-review PR is awaiting review from an assigned reviewer team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. labels Nov 11, 2023
@bazel-io bazel-io added the potential release blocker Flagged by community members using "@bazel-io flag". Should be added to a release blocker milestone label Nov 11, 2023
@keertk
Copy link
Member

keertk commented Nov 12, 2023

@bazel-io fork 7.0.0

@bazel-io bazel-io removed the potential release blocker Flagged by community members using "@bazel-io flag". Should be added to a release blocker milestone label Nov 12, 2023
@meteorcloudy meteorcloudy added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Nov 13, 2023
@github-actions github-actions bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Nov 13, 2023
bazel-io pushed a commit to bazel-io/bazel that referenced this pull request Nov 13, 2023
`BazelLockFileModule` is the only consumer of `BazelModuleResolutionEvent` and only subscribes with `--lockfile_mode=update`.

Closes bazelbuild#20154.

PiperOrigin-RevId: 581955314
Change-Id: I2ef8a39833c1d195eccdd32fd8e3f86782979dc5
keertk pushed a commit that referenced this pull request Nov 13, 2023
`BazelLockFileModule` is the only consumer of
`BazelModuleResolutionEvent` and only subscribes with
`--lockfile_mode=update`.

Closes #20154.

Commit
560327f

PiperOrigin-RevId: 581955314
Change-Id: I2ef8a39833c1d195eccdd32fd8e3f86782979dc5

Co-authored-by: Fabian Meumertzheim <fabian@meumertzhe.im>
@fmeum fmeum deleted the patch-7 branch November 14, 2023 17:39
@iancha1992
Copy link
Member

The changes in this PR have been included in Bazel 7.0.0 RC5. Please test out the release candidate and report any issues as soon as possible. If you're using Bazelisk, you can point to the latest RC by setting USE_BAZEL_VERSION=last_rc.
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants