Skip to content

Commit

Permalink
Propagate exit status of nix command
Browse files Browse the repository at this point in the history
Introduced in v1.2.0.

Type: fix
Issue: 22
  • Loading branch information
nicknovitski committed Nov 12, 2024
1 parent 1c99270 commit 214f200
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 2 deletions.
9 changes: 9 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,15 @@ jobs:
- uses: actions/checkout@v4
- uses: cachix/install-nix-action@v30
- uses: ./
- name: Make the nix command fail
uses: ./
with:
arguments: --a-bad-flag
id: should-fail
continue-on-error: true
- name: Fail if the action succeeded when the nix command failed
if: steps.should-fail.outcome == 'success'
run: exit 1
check:
strategy:
matrix:
Expand Down
5 changes: 3 additions & 2 deletions nix-develop-gha.sh
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,8 @@ contains() {
envOutput=

# Iterate over the output of `env -0` in the shell environment
while IFS='=' read -r -d '' n v; do
while IFS='=' read -r -d '' n v || exit "$n"
do
# If this variable is empty then we're in the first loop, and $n is the
# output of the shellHook _before_ `env` is run.
if ! [ "$envOutput" ]; then
Expand Down Expand Up @@ -68,4 +69,4 @@ while IFS='=' read -r -d '' n v; do
fi
# Add all environment variables except for PATH to GITHUB_ENV.
printf "%s=%s\n" "$n" "$v" >>"${GITHUB_ENV:-/dev/stderr}"
done < <(nix develop "${arguments[@]}" --command bash -c "echo -ne '\0'; env -0")
done < <(set +e; nix develop "${arguments[@]}" --command bash -c "echo -ne '\0'; env -0"; printf '%s' "$?")

0 comments on commit 214f200

Please sign in to comment.