From 95eb1c3a35f900d86516757dff499e8b3e3076dc Mon Sep 17 00:00:00 2001 From: Andrew Quenehen Date: Sun, 2 Aug 2026 21:53:19 -0300 Subject: [PATCH] ci(release): clarify Switch hard-fail vs path-gated PR CI Co-authored-by: Cursor --- .github/workflows/release.yml | 7 +++++-- tests/switch_ci_workflows_test.lua | 5 ++++- 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index e7c258ef..e7658c4e 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -200,8 +200,11 @@ jobs: - name: Build Switch run: | set -euo pipefail - # Fused NRO; needs native switch-tools (nacptool/elf2nro) and/or - # Docker on the Mac self-hosted runner; see docs/switch-build.md. + # Hard-fail gate: Switch ships with every release (never soft-fail). + # PR CI is path-gated (ubuntu selftest + canonical fused); release + # always builds Switch regardless of which files changed. + # Needs native switch-tools (nacptool/elf2nro) and/or Docker on the + # Mac self-hosted runner; see docs/switch-build.md. scripts/build_switch.sh --fetch --fused \ --version "${{ steps.ver.outputs.version }}" diff --git a/tests/switch_ci_workflows_test.lua b/tests/switch_ci_workflows_test.lua index 6b553080..b241431f 100644 --- a/tests/switch_ci_workflows_test.lua +++ b/tests/switch_ci_workflows_test.lua @@ -129,7 +129,10 @@ do local nextStep = rest:find("\n - name:", 2) local block = nextStep and rest:sub(1, nextStep - 1) or rest mustContain(block, "scripts/build_switch.sh --fetch --fused", "release Build Switch") - mustNotContain(block, "continue-on-error", "release Build Switch") + -- YAML key must be absent (comment prose may discuss soft-fail policy) + mustNotContain(block, "continue-on-error:", "release Build Switch") + mustContain(block, "path-gated", "release Build Switch comment") + mustContain(block, "Hard-fail", "release Build Switch comment") end T.finish("switch_ci_workflows_test")