-
-
Notifications
You must be signed in to change notification settings - Fork 651
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
Upgrade to Node v12 #3780
Comments
Current status (updated):
|
v10 will still be supported (as LTS) up to 2021-04-30. So I think it'd be OK to still be on it up to then, for as long as there's a reason to be. Definitely it's good to be on current versions of tools when we can, though. So once that blocker on Windows is resolved, we should upgrade unless there's some other blocker in the way. |
Also, as you may well know but for concreteness here in the thread: what it means that we're currently on Node v10 is
So upgrading is a matter of confirming that everything works on the new version, and then updating those two places. |
We learned today (chat thread) about one major improvement that comes with Node 12: the startup time goes from painful (like 350ms for Meanwhile, from looking at https://github.com/facebook/metro/releases , I learned that that Metro bug is also fixed in a stable release, Metro v0.56.4 (thanks, Metro maintainers!). And Metro v0.56 in turn will come with RN v0.61, #3781 . So that's a bit closer than v0.62, then, happily. |
We're now on RN v0.61! So the known Metro issue on Windows that was blocking this is resolved. The next step is for us to switch over in local development, and see if anything goes wrong. In particular I don't believe any of us have tested Node 12 on macOS yet. |
Both @chrisbobbe and I have been using Node 12 for our local development for a couple of months now (chat thread), and things seem good. So I'll go make this update. |
We've been using this for local development for a little while, and all seems well. The major visible benefit to this upgrade is that startup times are much improved: zulip#3780 (comment) which means everything we do from a script becomes much faster. Fixes: zulip#3780
At present (2020-01), Node v12 is the latest active LTS. We should probably be ready to move up to v12 before v10 goes into "Maintenance" mode in 2020-04.
The text was updated successfully, but these errors were encountered: