Skip to content

Pin CI actions and add zizmor and dependabot - #96

Merged
andrew merged 1 commit into
mainfrom
harden-ci
Oct 15, 2026
Merged

andrew merged 1 commit into
mainfrom
harden-ci

Conversation

@andrew

@andrew andrew commented Oct 15, 2026

Copy link
Copy Markdown
Member

Pins actions/checkout and actions/setup-node to commit SHAs, sets permissions: {} at the top level with contents: read on the job, and passes persist-credentials: false to checkout. Adds a zizmor workflow and a dependabot config for github-actions and npm with a 7 day cooldown.

Also adds Node 24 to the test matrix. Node 18 stays while engines is >=18.

Copilot AI balanced review requested due to automatic review settings October 15, 2026 08:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@andrew
andrew merged commit fa80c09 into main Oct 15, 2026
6 checks passed
@andrew
andrew deleted the harden-ci branch October 15, 2026 08:53
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.

3 participants