Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 34.2k
deps: update llhttp to 9.2.0#51719
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: update llhttp to 9.2.0 #51719
Uh oh!
There was an error while loading. Please reload this page.
Conversation
nodejs-github-bot commented Feb 11, 2024
Review requested:
|
mcollina left a comment
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
96e8ae6 to 5380ceeComparenodejs-github-bot commented Feb 14, 2024
nodejs-github-bot commented Feb 16, 2024
nodejs-github-bot commented Feb 19, 2024
5380cee to 630de1cComparenodejs-github-bot commented Mar 1, 2024
nodejs-github-bot commented Mar 7, 2024
630de1c to e31fd20Comparenodejs-github-bot commented Mar 12, 2024
targos commented Mar 12, 2024
Are there notable changes? |
marco-ippolito commented Mar 12, 2024
support for http method QUERY, @ShogunPanda anything else? |
ShogunPanda commented Mar 12, 2024
Nope, that's it! |
The notable-change Please suggest a text for the release notes if you'd like to include a more detailed summary, then proceed to update the PR description with the text or a link to the notable change suggested text comment. Otherwise, the commit will be placed in the Other Notable Changes section. |
e31fd20 to a8f0da6Comparenodejs-github-bot commented Mar 13, 2024
nodejs-github-bot commented Mar 13, 2024
Landed in 25a6fb6 |
PR-URL: nodejs#51719 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Ethan Arrowood <[email protected]>
PR-URL: nodejs#51719 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Ethan Arrowood <[email protected]>
marco-ippolito commented May 21, 2024 • edited
Loading Uh oh!
There was an error while loading. Please reload this page.
edited
Uh oh!
There was an error while loading. Please reload this page.
@nodejs/releasers is it ok to land this in v20.14.0 although a major bump? it adds support for http QUERY |
richardlau commented May 21, 2024
Probably not because #48981 is semver-major |
PR-URL: nodejs/node#51719 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Ethan Arrowood <[email protected]>
PR-URL: #51719 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Ethan Arrowood <[email protected]>
This is an automated update of llhttp to 9.2.0.