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

[7.2.0] Show display form labels in java_* buildozer fixups #21948

Closed
wants to merge 1 commit into from

Conversation

iancha1992
Copy link
Member

The change in #21702 only fixed the buildozer fixup messages for java_* actions entirely defined in Starlark. This commit makes the analogous change to the remaining native actions.

Related to #20486 and #21702

Closes #21827.

PiperOrigin-RevId: 623253302
Change-Id: Iadc2b0d8ce64f359b921ad7a7c489111a3a29997

Commit 1c99866

The change in bazelbuild#21702 only fixed the buildozer fixup messages for `java_*` actions entirely defined in Starlark. This commit makes the analogous change to the remaining native actions.

Related to bazelbuild#20486 and bazelbuild#21702

Closes bazelbuild#21827.

PiperOrigin-RevId: 623253302
Change-Id: Iadc2b0d8ce64f359b921ad7a7c489111a3a29997
@iancha1992 iancha1992 added team-Configurability platforms, toolchains, cquery, select(), config transitions team-Rules-Java Issues for Java rules awaiting-review PR is awaiting review from an assigned reviewer labels Apr 9, 2024
@iancha1992 iancha1992 requested a review from justinhorvitz April 9, 2024 21:20
@iancha1992 iancha1992 requested a review from a team as a code owner April 9, 2024 21:20
@iancha1992 iancha1992 enabled auto-merge April 9, 2024 21:20
@iancha1992
Copy link
Member Author

@justinhorvitz looks like the tests are failing. Could you please take a look?

cc: @bazelbuild/triage

@fmeum
Copy link
Collaborator

fmeum commented Apr 9, 2024

@iancha1992 It's not the tests, already the build is failing since this commit requires #21702.

@iancha1992
Copy link
Member Author

Duplicate. #21956

@iancha1992 iancha1992 closed this Apr 10, 2024
auto-merge was automatically disabled April 10, 2024 16:13

Pull request was closed

@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Apr 10, 2024
github-merge-queue bot pushed a commit that referenced this pull request Apr 10, 2024
This cherry-picks:
* Stringify `Label`s in display form in `Args`
(9d3a8b0)
* Show display form labels in `java_*` buildozer fixups
(1c99866)

Fixes #21948 
Fixes #21826
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Configurability platforms, toolchains, cquery, select(), config transitions team-Rules-Java Issues for Java rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants