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

fix: recognize explicitly empty version strings #175

Merged
merged 3 commits into from
Nov 12, 2024

Conversation

fmeum
Copy link
Member

@fmeum fmeum commented Nov 12, 2024

A module directive with version = "" resulted in the version being stamped into the first bazel_dep.

Also fix the indentation to follow buildifier's style.

@fmeum fmeum requested a review from kormide November 12, 2024 10:57
fmeum added a commit to bazel-contrib/bazel-gazelle that referenced this pull request Nov 12, 2024
The new style also isn't supported by Publish to BCR and a fix is in flight: bazel-contrib/publish-to-bcr#175
fmeum added a commit to bazel-contrib/bazel-gazelle that referenced this pull request Nov 12, 2024
**What type of PR is this?**

Bug fix

**What package or component does this PR mostly affect?**

BCR

**What does this PR do? Why is it needed?**

The new style also isn't supported by Publish to BCR and a fix is in
flight: bazel-contrib/publish-to-bcr#175

**Which issues(s) does this PR fix?**

Fixes #

**Other notes for review**
@kormide
Copy link
Collaborator

kormide commented Nov 12, 2024

Thanks! Added a commit to update the e2es.

@kormide
Copy link
Collaborator

kormide commented Nov 12, 2024

Please squash when merging.

@fmeum fmeum merged commit 0b382f4 into bazel-contrib:main Nov 12, 2024
2 checks passed
@fmeum fmeum deleted the fix-version-replacement branch November 13, 2024 22:00
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.

2 participants