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 crash when MODULE.bazel has foo = bar.baz() #1263

Merged
merged 1 commit into from Apr 17, 2024

Conversation

fmeum
Copy link
Contributor

@fmeum fmeum commented Apr 12, 2024

Such assignments don't do anything, but we shouldn't crash on them.

Context: https://bazelbuild.slack.com/archives/C014RARENH0/p1712945309438879

Such assignments don't do anything, but we shouldn't crash on them.
@vladmos vladmos merged commit 09e2763 into bazelbuild:master Apr 17, 2024
2 checks passed
@fmeum fmeum deleted the fix-tag-assignment branch April 17, 2024 17:35
copybara-service bot pushed a commit to bazelbuild/bazel that referenced this pull request Apr 19, 2024
Fixes bazelbuild/buildtools#1263

Closes #22045.

PiperOrigin-RevId: 626307672
Change-Id: Ib088e73835ea3d7076976a9de4ab4207ff491b27
iancha1992 pushed a commit to iancha1992/bazel that referenced this pull request Apr 19, 2024
Fixes bazelbuild/buildtools#1263

Closes bazelbuild#22045.

PiperOrigin-RevId: 626307672
Change-Id: Ib088e73835ea3d7076976a9de4ab4207ff491b27
iancha1992 pushed a commit to iancha1992/bazel that referenced this pull request Apr 22, 2024
Fixes bazelbuild/buildtools#1263

Closes bazelbuild#22045.

PiperOrigin-RevId: 626307672
Change-Id: Ib088e73835ea3d7076976a9de4ab4207ff491b27
github-merge-queue bot pushed a commit to bazelbuild/bazel that referenced this pull request Apr 24, 2024
Fixes bazelbuild/buildtools#1263

Closes #22045.

PiperOrigin-RevId: 626307672
Change-Id: Ib088e73835ea3d7076976a9de4ab4207ff491b27

Commit
7979dd6

---------

Co-authored-by: Fabian Meumertzheim <fabian@meumertzhe.im>
apattidb pushed a commit to databricks/buildtools that referenced this pull request May 10, 2024
Such assignments don't do anything, but we shouldn't crash on them.
Kila2 pushed a commit to Kila2/bazel that referenced this pull request May 13, 2024
Fixes bazelbuild/buildtools#1263

Closes bazelbuild#22045.

PiperOrigin-RevId: 626307672
Change-Id: Ib088e73835ea3d7076976a9de4ab4207ff491b27
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

2 participants