Skip to content

Commit 8875e8f

Browse files
radicalCopilot
andcommitted
Always build CLI E2E Docker image, independent of matrix gate
The build_cli_e2e_image reusable workflow was previously gated on `tests_matrix_requires_cli_archive` so it only ran when the test matrix contained at least one job that needed the CLI archive. That gate also forced the image build to wait for setup_for_tests to finish computing the matrix, which delayed test jobs that depend on the image. Drop the gate so the image build kicks off in parallel with setup. The extra cost on PRs that don't need the image is one short-lived Docker build job, which is preferable to serializing every CLI E2E test run on matrix setup. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 6366cca commit 8875e8f

1 file changed

Lines changed: 0 additions & 1 deletion

File tree

.github/workflows/tests.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,6 @@ jobs:
8686
name: Build CLI E2E Docker image
8787
uses: ./.github/workflows/build-cli-e2e-image.yml
8888
needs: setup_for_tests
89-
if: ${{ fromJson(needs.setup_for_tests.outputs.tests_matrix_requires_cli_archive).include[0] != null }}
9089

9190
tests_no_nugets:
9291
name: ${{ matrix.shortname }}

0 commit comments

Comments
 (0)