Skip to content

deps: patch V8 to 8.1.307.30 - #32693

Closed
targos wants to merge 2 commits into
nodejs:masterfrom
targos:v8-28
Closed

targos wants to merge 2 commits into
nodejs:masterfrom
targos:v8-28

Conversation

@targos

@targos targos commented Apr 6, 2020

Copy link
Copy Markdown
Member

@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

nodejs-github-bot commented Apr 12, 2020 •

Copy link
Copy Markdown
Collaborator

@targos targos added the author ready PRs with CI started, the required approvals, and no outstanding review comments. label Apr 12, 2020
@addaleax addaleax removed the author ready PRs with CI started, the required approvals, and no outstanding review comments. label Apr 13, 2020
@addaleax

Copy link
Copy Markdown
Member

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

@targos

targos commented Apr 13, 2020

Copy link
Copy Markdown
Member Author

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

@bricss

bricss commented Apr 15, 2020

Copy link
Copy Markdown
Contributor

Is it gonna drop into the node 14? 🥺

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@addaleax

Copy link
Copy Markdown
Member

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

@targos

targos commented Apr 16, 2020

Copy link
Copy Markdown
Member Author

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

@targos

targos commented Apr 18, 2020

Copy link
Copy Markdown
Member Author

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

nodejs-github-bot commented Apr 18, 2020 •

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

nodejs-github-bot commented Apr 20, 2020 •

Copy link
Copy Markdown
Collaborator

@targos

targos commented Apr 20, 2020

Copy link
Copy Markdown
Member Author

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