From dd04b46e86d04d72682164b4a766baf9ddef12dc Mon Sep 17 00:00:00 2001 From: Tommaso Sciortino Date: Wed, 7 Jan 2026 16:18:33 -0800 Subject: [PATCH] Fix CI for forks (#16113) --- .github/workflows/ci.yml | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 3fb1867db7..6b95abdddc 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -125,7 +125,7 @@ jobs: args: '--verbose --accept 200,503 ./**/*.md' fail: true test_linux: - name: 'Test (Linux) - ${{ matrix.shard }}' + name: 'Test (Linux) - ${{ matrix.node-version }}, ${{ matrix.shard }}' runs-on: 'gemini-cli-ubuntu-16-core' needs: - 'merge_queue_skipper' @@ -185,7 +185,7 @@ jobs: ${{ always() && (github.event.pull_request.head.repo.full_name == github.repository) }} uses: 'dorny/test-reporter@dc3a92680fcc15842eef52e8c4606ea7ce6bd3f3' # ratchet:dorny/test-reporter@v2 with: - name: 'Test Results (Node ${{ matrix.node-version }}, ${{ matrix.shard }})' + name: 'Test Results (Node ${{ runner.os }}, ${{ matrix.node-version }}, ${{ matrix.shard }})' path: 'packages/*/junit.xml' reporter: 'java-junit' fail-on-error: 'false' @@ -195,12 +195,12 @@ jobs: ${{ always() && (github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name != github.repository) }} uses: 'actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02' # ratchet:actions/upload-artifact@v4 with: - name: 'test-results-fork-${{ matrix.node-version }}-${{ runner.os }}' + name: 'test-results-fork-${{ runner.os }}-${{ matrix.node-version }}-${{ matrix.shard }}' path: 'packages/*/junit.xml' test_mac: - name: 'Test (Mac) - ${{ matrix.shard }}' - runs-on: '${{ matrix.os }}' + name: 'Test (Mac) - ${{ matrix.node-version }}, ${{ matrix.shard }}' + runs-on: 'macos-latest' needs: - 'merge_queue_skipper' if: "${{needs.merge_queue_skipper.outputs.skip == 'false'}}" @@ -211,8 +211,6 @@ jobs: continue-on-error: true strategy: matrix: - os: - - 'macos-latest' node-version: - '20.x' - '22.x' @@ -262,7 +260,7 @@ jobs: ${{ always() && (github.event.pull_request.head.repo.full_name == github.repository) }} uses: 'dorny/test-reporter@dc3a92680fcc15842eef52e8c4606ea7ce6bd3f3' # ratchet:dorny/test-reporter@v2 with: - name: 'Test Results (Node ${{ matrix.node-version }}, ${{ matrix.shard }})' + name: 'Test Results (Node ${{ runner.os }}, ${{ matrix.node-version }}, ${{ matrix.shard }})' path: 'packages/*/junit.xml' reporter: 'java-junit' fail-on-error: 'false' @@ -272,7 +270,7 @@ jobs: ${{ always() && (github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name != github.repository) }} uses: 'actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02' # ratchet:actions/upload-artifact@v4 with: - name: 'test-results-fork-${{ matrix.node-version }}-${{ runner.os }}' + name: 'test-results-fork-${{ runner.os }}-${{ matrix.node-version }}-${{ matrix.shard }}' path: 'packages/*/junit.xml' - name: 'Upload coverage reports' @@ -280,7 +278,7 @@ jobs: ${{ always() }} uses: 'actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02' # ratchet:actions/upload-artifact@v4 with: - name: 'coverage-reports-${{ matrix.node-version }}-${{ matrix.os }}' + name: 'coverage-reports-${{ runner.os }}-${{ matrix.node-version }}-${{ matrix.shard }}' path: 'packages/*/coverage' codeql: