-
Notifications
You must be signed in to change notification settings - Fork 577
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
Doing another Semver Minor on 8.x #364
Comments
/cc @nodejs/lts |
👍 |
+1 from me. It seems important for http2 adoption. |
Is there a specific part that we should likely also backport? I do have some time that I could invest into that at the moment. |
/cc @kjin who may be able to chime in on this. |
Current status -- the PR should be complete for backportable changes to http2 up to Node 10.9. I'm currently working through additional commits up to 10.11. It shouldn't be too hard to remove the semver-minor change to http in this PR... if that helps it land. |
We have approval for another minor. Let's get everything in that makes
sense!
…On Wed, Oct 3, 2018, 4:04 PM Kelvin Jin ***@***.***> wrote:
Current status -- the PR should be complete for backportable changes to
http2 up to Node 10.9. I'm currently working through additional commits up
to 10.11.
It shouldn't be too hard to remove the semver-minor change to http in this
PR... if that helps it land.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#364 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAecV2Rl2NGo1TJCeuME-fKtTMtTlQG0ks5uhRhHgaJpZM4XETBW>
.
|
Just updated nodejs/node#22850 |
Hey All,
We have an open PR to get the http2 implementation up to date on 8.x... this PR does include semver-minor changes to http. I'd like to propose we do another semver-minor release of the 8.x line
nodejs/node#22850
The text was updated successfully, but these errors were encountered: