Skip to content

Conversation

@Wyverald
Copy link
Member

@Wyverald Wyverald commented Jan 7, 2026

Work towards #28056

Closes #28061.

PiperOrigin-RevId: 852463750
Change-Id: I100a76adf564aa11dbef1a98f48a6b1389c1d584

Work towards #28056

Closes #28061.

PiperOrigin-RevId: 852463750
Change-Id: I100a76adf564aa11dbef1a98f48a6b1389c1d584
@Wyverald Wyverald requested a review from a team as a code owner January 7, 2026 17:08
@github-actions github-actions bot added team-Configurability platforms, toolchains, cquery, select(), config transitions awaiting-review PR is awaiting review from an assigned reviewer labels Jan 7, 2026
@iancha1992 iancha1992 enabled auto-merge January 7, 2026 17:09
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 the describeKey method for RepoMappingManifestAction, which will improve debuggability of action cache issues. The implementation correctly mirrors the logic in computeKey, providing a human-readable description of the components of the action key. The change also includes a refactoring of several MapFn fields to the more specific CommandLineItem.ExceptionlessMapFn, which is necessary for using the new describeNestedSetFingerprint helper and improves type safety. The changes are correct and well-implemented.

@iancha1992 iancha1992 added this pull request to the merge queue Jan 7, 2026
Merged via the queue into release-8.5.1 with commit c9cbff2 Jan 7, 2026
50 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Jan 7, 2026
@Wyverald Wyverald deleted the wyv-851-describekey branch January 7, 2026 19:08
iancha1992 pushed a commit to iancha1992/bazel that referenced this pull request Jan 8, 2026
…#28181)

Work towards bazelbuild#28056

Closes bazelbuild#28061.

PiperOrigin-RevId: 852463750
Change-Id: I100a76adf564aa11dbef1a98f48a6b1389c1d584

Co-authored-by: Fabian Meumertzheim <[email protected]>
github-merge-queue bot pushed a commit that referenced this pull request Jan 9, 2026
…28194)

Work towards #28056

Closes #28061.

PiperOrigin-RevId: 852463750
Change-Id: I100a76adf564aa11dbef1a98f48a6b1389c1d584

Commit
c9cbff2

Co-authored-by: Xùdōng Yáng <[email protected]>
Co-authored-by: Fabian Meumertzheim <[email protected]>
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.

4 participants