-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
Description
The Quality Gates workflow enforces pipefail in all bin scripts, but adding it naively breaks pipes that use | head due to SIGPIPE (exit 141).
Requires a careful refactor:
- Switch shebang from
#\!/bin/shto#\!/usr/bin/env bash(pipefail is not POSIX) - Add
set -o pipefailafterset -ein all 20 executable scripts - Handle SIGPIPE in all
| head,| sed ... | head, and similar truncating pipelines - Options:
trap '' PIPE,|| trueon specific pipes, or restructure to avoid early-exit readers
Affected pipelines (examples):
git log ... | sed | head -1in git-issue-show (state extraction)git for-each-ref ... | awk | sortin git-issue-ls- Similar patterns in most bin scripts (6,301 total lines, 400+ pipe usages)
Attempted quick fix (shebang + pipefail) caused exit 141 across core tests. Reverted.
Reactions are currently unavailable