FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

deps: patch V8 to 9.0.257.24 by targos · Pull Request #38423 · nodejs/node · GitHub

/ node Public

deps: patch V8 to 9.0.257.24 - #38423

Closed
targos wants to merge 1 commit into
nodejs:masterfrom
targos:v8-9.0.257.24
Closed

deps: patch V8 to 9.0.257.24#38423
targos wants to merge 1 commit into
nodejs:masterfrom
targos:v8-9.0.257.24

Conversation

targos commented Apr 26, 2021

Copy link
Copy Markdown
Member

github-actions Bot added needs-ci PRs that need a full CI run. v8 engine Issues and PRs related to the V8 dependency. labels Apr 26, 2021

This comment has been minimized.

This comment has been minimized.

nodejs-github-bot commented Apr 27, 2021
edited by jasnell
Loading

Copy link
Copy Markdown
Collaborator

targos added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Apr 27, 2021

jasnell commented Apr 28, 2021

Copy link
Copy Markdown
Member

Landed in 8b60369

jasnell closed this Apr 28, 2021
jasnell pushed a commit that referenced this pull request Apr 28, 2021
Refs: v8/v8@9.0.257.21...9.0.257.24

PR-URL: #38423
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos deleted the v8-9.0.257.24 branch April 28, 2021 20:48
targos added a commit that referenced this pull request Apr 29, 2021
Refs: v8/v8@9.0.257.21...9.0.257.24

PR-URL: #38423
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos mentioned this pull request May 3, 2021

zsims commented May 11, 2021

Copy link
Copy Markdown

@targos when will this land in a v14 LTS release? Per https://github.com/nodejs/Release there should be a minor semver patch?

Copy link
Copy Markdown
Contributor

@zsims is this what you're looking for? 6b115d7

Available in 14.17.0 as of 2021-05-11 (possibly right after your comment!)

zsims commented May 12, 2021

Copy link
Copy Markdown

@tchetwin Exactly what I'm looking for. Thanks for your work on this + the release :-)

zsims commented May 12, 2021

Copy link
Copy Markdown

@tchetwin I would have expected to see v8/v8@df40e08 (as in this PR). Will this be making it to v14 LTS?

zsims commented May 25, 2021

Copy link
Copy Markdown

@tchetwin any updates for when/if v8/v8@df40e08 (in this PR) will land in v14 LTS?

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. needs-ci PRs that need a full CI run. v8 engine Issues and PRs related to the V8 dependency.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL