-
Notifications
You must be signed in to change notification settings - Fork 30.3k
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
v18.14.2 proposal #46724
v18.14.2 proposal #46724
Conversation
Refs: v8/v8@10.2.154.23...10.2.154.26 PR-URL: #46446 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Jiawen Geng <[email protected]>
The `snapshot.blob` file landed while backporting 2c0f7d4 in the v18.x release line, but is not required Fixes: #46133 PR-URL: #46626 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Myles Borins <[email protected]>
PR-URL: #46353 Reviewed-By: Myles Borins <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
PR-URL: #46673 Reviewed-By: Luke Karrys <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Ruy Adorno <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Myles Borins <[email protected]>
PR-URL: #46711 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
This comment was marked as outdated.
This comment was marked as outdated.
The commit message of a1a9d91 says "PR-URL: TODO", this would need to be amended I think. |
a1a9d91
to
4df0001
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Tests are all green and nothing looks odd in CITGM. I'll do one last review of CITGM tomorrow before promoting the release. |
There is a newish failure with I'm going to move forward with the release |
2023-02-21, Version 18.14.2 'Hydrogen' (LTS), @MylesBorins
Notable Changes
f864bef32a
] - deps: upgrade npm to 9.5.0 (npm team) #46673Commits
880a65d7ff
] - build: deletesnapshot.blob
file from the project (Juan José Arboleda) #46626cbea56efda
] - deps: update undici to 5.20.0 (Node.js GitHub Bot) #46711f864bef32a
] - deps: upgrade npm to 9.5.0 (npm team) #46673648041d568
] - deps: upgrade npm to 9.4.0 (npm team) #463535e1f213f3c
] - deps: patch V8 to 10.2.154.26 (Michaël Zasso) #46446