Skip to content

Commit 71f5cdd

Browse files
authored
Merge pull request #213 from mojoatomic/ci/enforce-health-gate
ci: enforce health gate (remove continue-on-error; run ratchet:context normally)
2 parents f5dfe39 + 41ffca9 commit 71f5cdd

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

.github/workflows/ci-ratchet.yml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -29,17 +29,15 @@ jobs:
2929
run: npm run analyze:current
3030

3131
- name: Ratchet (fail on new debt)
32-
continue-on-error: ${{ github.event_name == 'pull_request' && startsWith(github.head_ref, 'feat/health-gating') }}
3332
run: npm run ratchet
3433

35-
- name: Context-aware ratchet (telemetry only)
36-
continue-on-error: true
34+
- name: Health gate (context-aware ratchet)
3735
run: |
3836
# Prefer script if available, otherwise fallback to direct invocation
3937
if npm run -s | grep -q "ratchet:context"; then
40-
npm run ratchet:context || true
38+
npm run ratchet:context
4139
else
42-
node scripts/ratchet.js --mode=context --baseline=analysis-baseline.json --current=analysis-current.json || true
40+
node scripts/ratchet.js --mode=context --baseline=analysis-baseline.json --current=analysis-current.json
4341
fi
4442
4543
- name: Run tests

0 commit comments

Comments
 (0)