Skip to content

feat(ci): add actionlint workflow validation and git hooks#102

Closed
kieronlanning wants to merge 2 commits into
mainfrom
feat/actionlint-hooks
Closed

feat(ci): add actionlint workflow validation and git hooks#102
kieronlanning wants to merge 2 commits into
mainfrom
feat/actionlint-hooks

Conversation

@kieronlanning

Copy link
Copy Markdown
Contributor

Adds actionlint-based workflow validation at every layer.

What

  • CI: new \�alidate-workflows\ job calls the reusable \kjldev/.github/.github/workflows/validate-workflows.yml\ workflow — runs actionlint on every PR, wired into \ci-gate\
  • Pre-commit hook (.husky/pre-commit): runs actionlint before every commit
  • Pre-push hook (.husky/pre-push): second check at push time
  • Both hooks skip gracefully with an install hint if actionlint isn't found
  • Install: \scoop install actionlint\

Why

Catches invalid action versions (e.g. \checkout@v6), bad expressions, and YAML errors locally before they reach GitHub.

kieronlanning and others added 2 commits May 24, 2026 17:10
- ci.yml: validate-workflows job calls kjldev/.github reusable workflow
  which runs actionlint on all .github/workflows/*.yml files
- Added to ci-gate so a bad workflow file blocks PR merge
- .husky/pre-commit: runs actionlint locally before every commit (gracefully
  skips if not installed, with install instructions)
- .husky/pre-push: same check at push time for extra safety
- husky v9 installed and configured via prepare script

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 24, 2026 16:10
@kieronlanning kieronlanning added the skip-changeset PR does not need a changeset entry label May 24, 2026
@kieronlanning kieronlanning enabled auto-merge (squash) May 24, 2026 16:11
@kieronlanning

Copy link
Copy Markdown
Contributor Author

Closing - removing husky dependency, will redo without it

auto-merge was automatically disabled May 24, 2026 16:11

Pull request was closed

@kieronlanning kieronlanning removed the request for review from Copilot May 24, 2026 16:34
@kieronlanning kieronlanning deleted the feat/actionlint-hooks branch May 25, 2026 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changeset PR does not need a changeset entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant