Skip to content

Conversation

@iancha1992
Copy link
Member

Work towards #28056

Closes #28061.

PiperOrigin-RevId: 852463750
Change-Id: I100a76adf564aa11dbef1a98f48a6b1389c1d584

Commit c9cbff2

…#28181)

Work towards bazelbuild#28056

Closes bazelbuild#28061.

PiperOrigin-RevId: 852463750
Change-Id: I100a76adf564aa11dbef1a98f48a6b1389c1d584

Co-authored-by: Fabian Meumertzheim <[email protected]>
@iancha1992 iancha1992 requested a review from Wyverald January 8, 2026 23:00
@iancha1992 iancha1992 requested a review from a team as a code owner January 8, 2026 23:00
@iancha1992 iancha1992 added team-Configurability platforms, toolchains, cquery, select(), config transitions awaiting-review PR is awaiting review from an assigned reviewer labels Jan 8, 2026
@iancha1992 iancha1992 enabled auto-merge January 8, 2026 23:00
Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request implements describeKey for RepoMappingManifestAction to improve action key debugging. The implementation correctly mirrors the fields used in computeKey. Additionally, the code is refactored to use CommandLineItem.ExceptionlessMapFn instead of the more generic MapFn, which improves type safety. The changes are correct and improve both debuggability and code quality.

@iancha1992 iancha1992 added this pull request to the merge queue Jan 9, 2026
Merged via the queue into bazelbuild:release-8.6.0 with commit 50b179c Jan 9, 2026
47 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Jan 9, 2026
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants