Skip to content

Commit aa83d1c

Browse files
hyperpolymathclaude
andcommitted
fix(ci): use || to capture exit code before set -e kills the script
GitHub Actions run blocks use set -e; the previous EXIT=$? was never reached because mix test --cover exited 3 first. Use || to capture it. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 35a42a7 commit aa83d1c

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

.github/workflows/elixir-ci.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,11 +37,10 @@ jobs:
3737

3838
- name: Run tests
3939
run: |
40-
mix test --cover
41-
EXIT=$?
40+
mix test --cover || EXIT=$?
4241
# Exit 3 = coverage threshold not met; acceptable while integration
4342
# tests are excluded (no WASM binary in CI). Tests still fail on 1/2.
44-
[ $EXIT -eq 0 ] || [ $EXIT -eq 3 ] || exit $EXIT
43+
[ "${EXIT:-0}" -eq 3 ] || exit "${EXIT:-0}"
4544
4645
- name: Security check
4746
run: mix deps.audit || true

0 commit comments

Comments
 (0)