Skip to content

[Please ignore] Do not store the repository name in RepoSpec#21206

Closed
iancha1992 wants to merge 4 commits intobazelbuild:release-7.1.0from
iancha1992:cp21205_1
Closed

[Please ignore] Do not store the repository name in RepoSpec#21206
iancha1992 wants to merge 4 commits intobazelbuild:release-7.1.0from
iancha1992:cp21205_1

Conversation

@iancha1992
Copy link
Copy Markdown
Member

The name can instead be set centrally in BzlmodRepoRuleFunction. This removes some unnecessary glue code and paves the way for rewriting canonical repo names based on knowledge of the full dep graph.

Work towards #20997

Closes #21026.

Commit a87a8db

PiperOrigin-RevId: 604256186
Change-Id: Ia49123f016d1512074b04ec458f017f4bbb88233

fmeum added 2 commits February 5, 2024 13:45
The name can instead be set centrally in `BzlmodRepoRuleFunction`. This removes some unnecessary glue code and paves the way for rewriting canonical repo names based on knowledge of the full dep graph.

Work towards bazelbuild#20997

Closes bazelbuild#21026.

PiperOrigin-RevId: 604256186
Change-Id: Ia49123f016d1512074b04ec458f017f4bbb88233
@iancha1992 iancha1992 requested a review from a team as a code owner February 5, 2024 21:50
@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 Feb 5, 2024
@iancha1992 iancha1992 removed team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. awaiting-review PR is awaiting review from an assigned reviewer labels Feb 5, 2024
@iancha1992 iancha1992 closed this Feb 5, 2024
@iancha1992 iancha1992 removed the request for review from a team February 5, 2024 22:09
@iancha1992 iancha1992 changed the title [7.1.0] Do not store the repository name in RepoSpec [Please ignore] Do not store the repository name in RepoSpec Feb 5, 2024
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