Skip to content

Conversation

@Wyverald
Copy link
Member

@Wyverald Wyverald commented Jan 7, 2026

This change updates the default value of the --incompatible_enable_proto_toolchain_resolution flag to true for Bazel builds.

The flag's default value and name are now defined in FlagConstants, with different values for the open-source and internal versions.

PiperOrigin-RevId: 844684939
Change-Id: I264db1921e52e885d1e211f2a627d42ce380bd41

… in Bazel.

This change updates the default value of the --incompatible_enable_proto_toolchain_resolution flag to true for Bazel builds.

The flag's default value and name are now defined in FlagConstants, with different values for the open-source and internal versions.

PiperOrigin-RevId: 844684939
Change-Id: I264db1921e52e885d1e211f2a627d42ce380bd41
@Wyverald Wyverald requested a review from a team as a code owner January 7, 2026 19:07
@github-actions github-actions bot added team-Rules-Java Issues for Java rules awaiting-review PR is awaiting review from an assigned reviewer labels Jan 7, 2026
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 updates the default value of the --incompatible_enable_proto_toolchain_resolution flag to true. The change is implemented by centralizing the flag's default value and name in FlagConstants, which is a good practice for managing flags that may have different defaults in different environments. The related test StarlarkJavaLiteProtoLibraryTest is correctly updated to explicitly set the old flag value to maintain its behavior. The changes are clean, correct, and follow existing patterns in the codebase. I have no further comments.

@Wyverald Wyverald added this to the 9.0.0 release blockers milestone Jan 7, 2026
@iancha1992 iancha1992 added this pull request to the merge queue Jan 8, 2026
Merged via the queue into release-9.0.0 with commit b43bf61 Jan 8, 2026
49 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Jan 8, 2026
@Wyverald Wyverald deleted the wyv-900-prototoolchain branch January 8, 2026 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Rules-Java Issues for Java rules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants