Skip to content
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.

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

deps: [email protected] for v8 release #6148

Closed
wants to merge 1 commit into from
Closed

deps: [email protected] for v8 release #6148

wants to merge 1 commit into from

Conversation

MickVanDuijn
Copy link

https://www.cve.org/CVERecord?id=CVE-2022-25881

Replicated the changes made by #6115 for the v8 release branch

@MickVanDuijn MickVanDuijn requested a review from a team as a code owner February 8, 2023 08:19
@MickVanDuijn MickVanDuijn requested review from wraithgar and removed request for a team February 8, 2023 08:19
@eugenpodaru
Copy link

+1

npm audit currently fails our pipeline due to this

@timmccarron
Copy link

MERGE IT!

@MickVanDuijn
Copy link
Author

MERGE IT!

still needs one more review 😇 and someone with Merge access

@chanix95
Copy link

+1

This is causing npm audit failures also in our end. Will remain alert for this merge.

Thanks!

@MickVanDuijn
Copy link
Author

@wraithgar I see that Github auto-assigned you, is there anything you can do here, or do you know who to involve to speed up merging this PR?

@wraithgar
Copy link
Member

@wraithgar wraithgar closed this Feb 14, 2023
@MickVanDuijn
Copy link
Author

We will update this soon.

https://github.com/npm/cli/blob/latest/CONTRIBUTING.md#dependencies

Thanks! Looking forward to the update then! 🙌

@wraithgar
Copy link
Member

We will update this soon.
https://github.com/npm/cli/blob/latest/CONTRIBUTING.md#dependencies

Thanks! Looking forward to the update then! 🙌

It happened yesterday

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.

5 participants