Skip to content

ci(workflow-linter): scope uses: grep to require leading whitespace - #12

Merged
hyperpolymath merged 1 commit into
mainfrom
claude/workflow-linter-grep-scoped-2026-05-26
May 27, 2026
Merged

hyperpolymath merged 1 commit into
mainfrom
claude/workflow-linter-grep-scoped-2026-05-26

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Fixes the self-referential false positive where the in-tree workflow-linter ran grep -rn "uses:" across workflow files including itself, matching its own comment lines and shell command lines.

Fix: grep -rn "uses:" → grep -rnE "^[[:space:]]+uses:" (matches standards/governance-reusable.yml line 640).

Part of the otpiser#11 estate-wide blocker sweep.

The in-tree workflow-linter.yml ran `grep -rn "uses:"` across
workflow files, which matched its own comment lines (`# Pattern:
uses: ...`) and shell command lines, causing a self-referential
false positive.

Fix: require leading whitespace before `uses:` (matches
standards/governance-reusable.yml line 640 pattern).
@hyperpolymath
hyperpolymath enabled auto-merge (squash) May 26, 2026 13:10
@hyperpolymath
hyperpolymath merged commit b6930d1 into main May 27, 2026
0 of 22 checks passed
@hyperpolymath
hyperpolymath deleted the claude/workflow-linter-grep-scoped-2026-05-26 branch May 29, 2026 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant