From 8044bf0206c12dfba4dd2169a66d1dbf978c02b8 Mon Sep 17 00:00:00 2001 From: ethernet Date: Wed, 10 Jun 2026 13:23:14 -0400 Subject: [PATCH] fix(ci): only save test durations when tests pass The save-durations job used `if: always()` which meant it would run even when the test matrix failed, potentially caching duration data from a failed/incomplete run. Changed to check needs.test.result == 'success' so durations are only cached when all test slices pass cleanly. --- .github/workflows/tests.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index 1b255abddc..a6e7738fa4 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -129,7 +129,7 @@ jobs: # (including PRs) get balanced slicing. save-durations: needs: test - if: always() && github.ref == 'refs/heads/main' + if: needs.test.result == 'success' && github.ref == 'refs/heads/main' runs-on: ubuntu-latest steps: - name: Download all slice durations @@ -219,4 +219,4 @@ jobs: env: OPENROUTER_API_KEY: "" OPENAI_API_KEY: "" - NOUS_API_KEY: "" + NOUS_API_KEY: "" \ No newline at end of file