Skip to content

Run npm audit fix.#132

Open
samcunliffe wants to merge 2 commits into
mainfrom
sc/manual-npm-audit-fix
Open

Run npm audit fix.#132
samcunliffe wants to merge 2 commits into
mainfrom
sc/manual-npm-audit-fix

Conversation

@samcunliffe
Copy link
Copy Markdown
Member

No description provided.

@samcunliffe samcunliffe requested a review from a team as a code owner March 31, 2026 06:39
@samcunliffe samcunliffe added dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code labels Mar 31, 2026
@samcunliffe samcunliffe enabled auto-merge (squash) March 31, 2026 06:40
Copy link
Copy Markdown
Member

@dpshelio dpshelio left a comment

Choose a reason for hiding this comment

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

I don't know how to review this 🫣

@samcunliffe
Copy link
Copy Markdown
Member Author

I don't know how to review this 🫣

You don't really. If it builds, then the security warnings should disappear.

@samcunliffe
Copy link
Copy Markdown
Member Author

OK. For some reason this job now takes > 1h which means the app tokens expire.

@samcunliffe samcunliffe force-pushed the sc/manual-npm-audit-fix branch from 4bc8326 to 2b0d75e Compare April 13, 2026 13:43
@samcunliffe
Copy link
Copy Markdown
Member Author

We'll need to wait for #141

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

Labels

dependencies Pull requests that update a dependency file javascript Pull requests that update javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants