-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
node12 is EOL #19451
node12 is EOL #19451
Conversation
I would revert because there is no necessity to require v14 and some OS are still shipping very old Node versions. For example, Ubuntu 22.04 ships with the v12 branch, they probably won't update to new major versions during the lifetime of that version. Also, the update missed updating version in at least in README, Makefile. |
IMO this PR is correct (or, at least, not wrong)
It doesn't miss anything, instead, it just makes everything correct (and up-to-dated). For example, in README, it already say that Gitea requires "Node.js LTS" and the link points to Node.js 16 LTS. For Makefile, it just checks package.json And for package.json, it already declared I don't know what's the necessary to keep using an outdated nodejs shipped with OS. For example, Debian 11 (stable) only has Golang 1.15, do you want to make the code can be compiled with Go 1.15? |
Right, I forgot we had it that way already.
My point is we should declare our minimum supported version and that remains v12.17 or greater. There's no strict need for v14 now and v12 will still see a lot of usage even if unsupported because it comes in Ubuntu/Debian base repos. IIRC, the Ubuntu/Debian maintainers will still backport critical fixes into those "forked" versions so the remain "supported" in a sense, just not by Node.js team. |
* giteaofficial/main: Fix logging of Transfer API (go-gitea#19456) RepoAssignment ensure to close before overwrite (go-gitea#19449) node12 is EOL (go-gitea#19451) Add Changelog v1.16.6 (go-gitea#19339) (go-gitea#19450) Fix DELETE request for non-existent public key (go-gitea#19443) [skip ci] Updated translations via Crowdin Don't panic on `ErrEmailInvalid` (go-gitea#19441)
* node12 is EOL * fix lockfile
https://endoflife.date/nodejs