From 8a64742ddf0e13d1c43e2ae8b7f964e9a787671a Mon Sep 17 00:00:00 2001 From: Colin McDonnell Date: Sat, 14 Feb 2026 02:10:18 +0000 Subject: [PATCH] sync action workflow fail-fast to match root workflow the root workflow was updated to fail-fast: true but the action workflow wasn't updated to match. the ci consistency test enforces they stay in sync. Co-authored-by: Cursor --- .github/workflows/test.yml | 4 ++-- test/ci.test.ts | 12 ++++++------ 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index b7a0c7e..1af9100 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -25,7 +25,7 @@ jobs: contents: read id-token: write strategy: - fail-fast: false + fail-fast: true matrix: agent: [claude, codex, cursor, gemini, opencode] test: @@ -55,7 +55,7 @@ jobs: contents: read id-token: write strategy: - fail-fast: false + fail-fast: true matrix: test: [ diff --git a/test/ci.test.ts b/test/ci.test.ts index 0c32053..ce31af6 100644 --- a/test/ci.test.ts +++ b/test/ci.test.ts @@ -114,9 +114,9 @@ describe("ci workflow consistency", () => { expect(getEnvVarNames(rootJob)).toEqual(expectedAgentEnvVars); }); - it("fail-fast is disabled in both", () => { - expect(rootJob.strategy!["fail-fast"]).toBe(false); - expect(actionJob.strategy!["fail-fast"]).toBe(false); + it("fail-fast is enabled in both", () => { + expect(rootJob.strategy!["fail-fast"]).toBe(true); + expect(actionJob.strategy!["fail-fast"]).toBe(true); }); }); @@ -148,9 +148,9 @@ describe("ci workflow consistency", () => { expect(getEnvVarNames(rootJob)).toEqual(expectedAgnosticEnvVars); }); - it("fail-fast is disabled in both", () => { - expect(rootJob.strategy!["fail-fast"]).toBe(false); - expect(actionJob.strategy!["fail-fast"]).toBe(false); + it("fail-fast is enabled in both", () => { + expect(rootJob.strategy!["fail-fast"]).toBe(true); + expect(actionJob.strategy!["fail-fast"]).toBe(true); }); }); });