Skip to content

[CI] Add prek audit hooks#409

Merged
jbampton merged 2 commits intoNextCommunity:mainfrom
jbampton:audit-workflows
Apr 4, 2026
Merged

[CI] Add prek audit hooks#409
jbampton merged 2 commits intoNextCommunity:mainfrom
jbampton:audit-workflows

Conversation

@jbampton
Copy link
Copy Markdown
Member

@jbampton jbampton commented Apr 4, 2026

closes #384

@jbampton jbampton self-assigned this Apr 4, 2026
@jbampton jbampton added this to Next Apr 4, 2026
@jbampton jbampton added this to the Hackfest milestone Apr 4, 2026
@github-project-automation github-project-automation bot moved this to In progress in Next Apr 4, 2026
@jbampton jbampton moved this from In progress to Review in progress in Next Apr 4, 2026
@jbampton jbampton added github_actions Pull requests that update Github_actions code tests yaml labels Apr 4, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request migrates the npm-audit hook from the main .pre-commit-config.yaml to a new dedicated configuration file, .pre-commit-config-audit.yaml. Feedback indicates that the exclusion of package-lock.json in the new configuration should be removed to ensure audits run on dependency updates, and the Node.js version should be corrected to a valid LTS version as 24.14.0 does not exist.

Comment thread .pre-commit-config-audit.yaml Outdated
Comment thread .pre-commit-config-audit.yaml
Comment thread .pre-commit-config-audit.yaml Outdated
@jbampton jbampton merged commit 18b73d1 into NextCommunity:main Apr 4, 2026
9 of 10 checks passed
@jbampton jbampton deleted the audit-workflows branch April 4, 2026 06:29
@github-project-automation github-project-automation bot moved this from Review in progress to Done in Next Apr 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update Github_actions code tests yaml

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Put npm hooks like npm audit in separate config file

1 participant