Skip to content

Conversation

@Pavank1992
Copy link
Contributor

ModuleExtensionMetadata incorrectly identified repos by their local names rather than the names used by the generating extension, which resulted in incorrect fixup warnings when supplying keyword arguments to use_repo.

Closes #18528.
commit 0af70cd

PiperOrigin-RevId: 536386347
Change-Id: I4368b0dcdc28d6b2827b74b3b0b73c31a0215c0f

ModuleExtensionMetadata incorrectly identified repos by their local names rather than the names used by the generating extension, which resulted in incorrect fixup warnings when supplying keyword arguments to `use_repo`.

Closes #18528.

PiperOrigin-RevId: 536386347
Change-Id: I4368b0dcdc28d6b2827b74b3b0b73c31a0215c0f
@Pavank1992 Pavank1992 requested a review from keertk as a code owner May 30, 2023 14:23
@Pavank1992 Pavank1992 requested a review from meteorcloudy May 30, 2023 14:25
@Pavank1992 Pavank1992 added team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. awaiting-review PR is awaiting review from an assigned reviewer labels May 30, 2023
Copy link
Member

@meteorcloudy meteorcloudy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@keertk keertk enabled auto-merge (squash) May 30, 2023 14:38
@keertk keertk merged commit 1289453 into bazelbuild:release-6.2.1 May 30, 2023
copybara-service bot pushed a commit that referenced this pull request Jun 2, 2023
Baseline:  758b44d

Release Notes:

+ Automatic code cleanup. (#18504)
+ Update relnotes scripts (#18503)
+ Fix #18493. (#18512)
+ Fix formatting of release notes (#18533)
+ Use extension rather than local names in ModuleExtensionMetadata (#18535)

Acknowledgements:

This release contains contributions from many people at Google, as well as Fabian Meumertzheim, Pavan Singh.
chiragramani pushed a commit to uber-common/bazel that referenced this pull request Jun 2, 2023
Baseline:  758b44d

Release Notes:

+ Automatic code cleanup. (bazelbuild#18504)
+ Update relnotes scripts (bazelbuild#18503)
+ Fix bazelbuild#18493. (bazelbuild#18512)
+ Fix formatting of release notes (bazelbuild#18533)
+ Use extension rather than local names in ModuleExtensionMetadata (bazelbuild#18535)

Acknowledgements:

This release contains contributions from many people at Google, as well as Fabian Meumertzheim, Pavan Singh.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review PR is awaiting review from an assigned reviewer team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants