Skip to content

Conversation

@fmeum
Copy link
Collaborator

@fmeum fmeum commented Dec 17, 2024

The warnings tell the user to run bazel mod tidy, which is very confusing.

Fixes #24495

The warnings tell the user to run `bazel mod tidy`, which is very confusing.
@fmeum fmeum marked this pull request as ready for review December 17, 2024 21:47
@github-actions github-actions bot added team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. awaiting-review PR is awaiting review from an assigned reviewer labels Dec 17, 2024
@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 Dec 18, 2024
@fmeum
Copy link
Collaborator Author

fmeum commented Dec 18, 2024

@bazel-io fork 8.0.1

@fmeum
Copy link
Collaborator Author

fmeum commented Dec 18, 2024

@bazel-io fork 7.5.0

@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 Dec 19, 2024
bazel-io pushed a commit to bazel-io/bazel that referenced this pull request Dec 19, 2024
The warnings tell the user to run `bazel mod tidy`, which is very confusing.

Fixes bazelbuild#24495

Closes bazelbuild#24729.

PiperOrigin-RevId: 708007222
Change-Id: I60dc889281a776bbf08a7f9537272d7692cce1d8
@fmeum fmeum deleted the 24495-warning branch December 20, 2024 11:13
github-merge-queue bot pushed a commit that referenced this pull request Dec 20, 2024
The warnings tell the user to run `bazel mod tidy`, which is very
confusing.

Fixes #24495

Closes #24729.

PiperOrigin-RevId: 708007222
Change-Id: I60dc889281a776bbf08a7f9537272d7692cce1d8

Commit
455ddb7

Co-authored-by: Fabian Meumertzheim <[email protected]>
fmeum added a commit to fmeum/bazel that referenced this pull request Jan 7, 2025
The warnings tell the user to run `bazel mod tidy`, which is very confusing.

Fixes bazelbuild#24495

Closes bazelbuild#24729.

PiperOrigin-RevId: 708007222
Change-Id: I60dc889281a776bbf08a7f9537272d7692cce1d8
(cherry picked from commit 455ddb7)
fmeum added a commit to fmeum/bazel that referenced this pull request Jan 7, 2025
The warnings tell the user to run `bazel mod tidy`, which is very confusing.

Fixes bazelbuild#24495

Closes bazelbuild#24729.

PiperOrigin-RevId: 708007222
Change-Id: I60dc889281a776bbf08a7f9537272d7692cce1d8
(cherry picked from commit 455ddb7)
meteorcloudy pushed a commit that referenced this pull request Jan 8, 2025
The warnings tell the user to run `bazel mod tidy`, which is very
confusing.

Fixes #24495

Closes #24729.

PiperOrigin-RevId: 708007222
Change-Id: I60dc889281a776bbf08a7f9537272d7692cce1d8 
(cherry picked from commit 455ddb7)

Fixes #24740
fmeum added a commit to fmeum/bazel that referenced this pull request Jan 29, 2025
The warnings tell the user to run `bazel mod tidy`, which is very confusing.

Fixes bazelbuild#24495

Closes bazelbuild#24729.

PiperOrigin-RevId: 708007222
Change-Id: I60dc889281a776bbf08a7f9537272d7692cce1d8
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.

bazel mod tidy logs a warning for the very thing it's about to fix

2 participants