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: patch V8 to 8.1.307.30 #32693

Closed
wants to merge 2 commits into from
Closed

deps: patch V8 to 8.1.307.30 #32693

wants to merge 2 commits into from

Conversation

targos
Copy link
Member

@targos targos commented Apr 6, 2020

@nodejs-github-bot nodejs-github-bot added the v8 engine Issues and PRs related to the V8 dependency. label Apr 6, 2020
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Apr 12, 2020

@targos targos added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 12, 2020
@addaleax addaleax removed the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 13, 2020
@addaleax
Copy link
Member

@targos I don’t think V8 CI is supposed to be all red, right?

@targos
Copy link
Member Author

targos commented Apr 13, 2020

@addaleax unfortunately, it currently is: nodejs/build#2256

@bricss
Copy link

bricss commented Apr 15, 2020

Is it gonna drop into the node 14? 🥺

@nodejs-github-bot
Copy link
Collaborator

@addaleax
Copy link
Member

@targos CI is green, I guess that means we should land it?

@targos
Copy link
Member Author

targos commented Apr 16, 2020

Yes, this can land. I'll do it tomorrow if nobody beats me to it

@targos
Copy link
Member Author

targos commented Apr 18, 2020

Rebased on master to unbreak V8 CI and updated to 8.1.307.30

@targos targos changed the title deps: patch V8 to 8.1.307.28 deps: patch V8 to 8.1.307.30 Apr 18, 2020
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Apr 18, 2020

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Apr 20, 2020

@targos
Copy link
Member Author

targos commented Apr 20, 2020

Landed in 3664807

targos added a commit that referenced this pull request Apr 20, 2020
PR-URL: #32693
Refs: v8/v8@8.1.307.26...8.1.307.28
Reviewed-By: Matheus Marchini <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Beth Griggs <[email protected]>
@targos targos closed this Apr 20, 2020
@targos targos deleted the v8-28 branch April 20, 2020 12:13
BethGriggs pushed a commit that referenced this pull request Apr 20, 2020
PR-URL: #32693
Refs: v8/v8@8.1.307.26...8.1.307.28
Reviewed-By: Matheus Marchini <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Beth Griggs <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v8 engine Issues and PRs related to the V8 dependency.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants