Skip to content

Conversation

@bazel-io
Copy link
Member

This avoids action conflicts when testonly non-test targets depend on test targets.

Until --incompatible_check_testonly_for_output_files is flipped, it is still possible to get action conflicts when a non-testonly target depends on an output file of a test. Since this can be fixed by adding the missing testonly annotation, this is only a minor concern.

Fixes #28056

Closes #28073.

PiperOrigin-RevId: 847892897
Change-Id: I60e908a22b51c307306760aa054b3968cfbebac2

Commit bcd8bcb

This avoids action conflicts when `testonly` non-test targets depend on test targets.

Until `--incompatible_check_testonly_for_output_files` is flipped, it is still possible to get action conflicts when a non-`testonly` target depends on an output file of a test. Since this can be fixed by adding the missing `testonly` annotation, this is only a minor concern.

Fixes bazelbuild#28056

Closes bazelbuild#28073.

PiperOrigin-RevId: 847892897
Change-Id: I60e908a22b51c307306760aa054b3968cfbebac2
@bazel-io bazel-io requested a review from a team as a code owner December 22, 2025 22:56
@bazel-io bazel-io added team-Configurability platforms, toolchains, cquery, select(), config transitions awaiting-review PR is awaiting review from an assigned reviewer labels Dec 22, 2025
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

The pull request changes the default value of the experimental_retain_test_configuration_across_testonly option in TestConfiguration.java from false to true. There were no review comments provided for these changes.

@iancha1992 iancha1992 enabled auto-merge December 23, 2025 00:10
@iancha1992 iancha1992 added this pull request to the merge queue Jan 7, 2026
Merged via the queue into bazelbuild:release-9.0.0 with commit f227566 Jan 7, 2026
46 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Jan 7, 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.

4 participants