Skip to content

Fix npm audit issues #258

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix npm audit issues #258

wants to merge 1 commit into from

Conversation

@paulcdejean
Copy link

Fixes #257

@paulcdejean
Copy link
Author

Looking into mocha issues now.

@paulcdejean
Copy link
Author

read-pkg 6 switched to esm which is one point of pain.

Another point of pain, is that there's failing tests even in the master branch.

@soluml soluml mentioned this pull request Nov 18, 2024
Open
@LucasPadovanPeek
Copy link

Can we get this reviewed/merged?
@mysticatea

@paulcdejean we could probably bump cross-spawn to 7.0.5

@paulcdejean
Copy link
Author

repo is dead unfortunately.

@paulcdejean
Copy link
Author

I decided not to use this package and instead just do a lot of && by hand.

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

Reviewers

2 more reviewers

@soluml soluml soluml approved these changes

@coliff coliff coliff approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

npm-run-all is reported as having a moderate severity vulnerabilty

4 participants

@paulcdejean @LucasPadovanPeek @soluml @coliff