Skip to content

ci: add actionlint to pre-commit checks - #615

Open
RowletCC wants to merge 1 commit into
NextCommunity:mainfrom
RowletCC:add-actionlint-pre-commit-373
Open

RowletCC wants to merge 1 commit into
NextCommunity:mainfrom
RowletCC:add-actionlint-pre-commit-373

Conversation

@RowletCC

Copy link
Copy Markdown

Closes #373.

Adds the upstream rhysd/actionlint pre-commit hook at the pinned v1.7.12 commit. It checks files under .github/workflows/ through the repository's existing Prek workflow. The optional ShellCheck/Pyflakes integrations are disabled for consistent results across the macOS, Linux, and Windows matrix; the repository's existing lint hooks remain in place.

Validation: prek run actionlint --all-files passed. I also ran the verified v1.7.12 release binary against every current workflow and checked the diff for whitespace errors.

@deepsource-io

deepsource-io Bot commented Sep 25, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in c36dd6c...8ad62c6 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
JavaScript Sep 25, 2026 9:33p.m. Review ↗
Secrets Sep 25, 2026 9:33p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@jbampton jbampton self-assigned this Sep 25, 2026
@jbampton jbampton added this to the Hacktoberfest 2026 milestone Sep 25, 2026
@jbampton jbampton added good first issue Good for newcomers tests yaml pre_commit Pull requests that update pre_commit code infra labels Sep 25, 2026
@jbampton
jbampton requested a lite review from Copilot September 25, 2026 21:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

No unresolved issues were identified, and validation passed.

Review effort: Lite
Findings: None

What changed in this PR

Adds a pinned actionlint pre-commit hook for GitHub Actions workflow validation.

Changes:

  • Pins actionlint v1.7.12.
  • Disables optional ShellCheck and Pyflakes integrations.
  • Restricts checks to workflow YAML files.
File Description
.pre-commit-config.yaml Registers the pinned actionlint hook.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

good first issue Good for newcomers infra pre_commit Pull requests that update pre_commit code tests yaml

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

Add actionlint with pre-commit

3 participants