-
Notifications
You must be signed in to change notification settings - Fork 107
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
Minimum Node.js version #169
Comments
I vote for picking a Node.js version that is not (yet) EOL (v16?) or is there a reason to pick v14 specifically? |
Mostly it's just kind of being a bit generous to anyone who may be using Dynalite but that hasn't yet upgraded to >14 (which I admit is probably not many folks). Also, AWS Lambda still supports Node.js 14 for a bit longer, so while it's an obscure use case, we have indeed run Dynalite in Lambda before. (That said, 16 also sounds fine to me!) |
(While I did merge that PR with 14 set as the minimum, I'd still like to keep this conversation open a bit longer, as we won't be releasing 4.0 immediately.) |
AWS is telling me node 14 is deprecated on their end by November 27, 2023. No issues jumping to node 16 for my projects. |
Sounds good to me! |
As mentioned in #166, we will be taking some time to onboard with the Dynalite codebase. As an early exercise, I've been taking a look at getting our dependencies up to date, namely because:
Level project
is on track to deprecate and transition a number of dependencies, including:levelup
(in favor ofabstract level
)leveldown
(in favor ofclassic-level
)memdown
(in favor ofmemory-level
)I haven't yet done a deeper analysis on the deprecations, but I figured we'd at least be better off getting things up to date so we aren't diffing very old versions against their eventual replacements.
Overall I haven't yet noticed any impact to the version updates to the Level dependencies, although they do deprecate support for Node.js versions <= 8 (see:
leveldown
changelog,memdown
changelog; see alsosubleveldown
6.0 upgrade guide).Because Dynalite doesn't have
package.json
engines
property, these updates will necessitate a breaking release (e.g. Dynalite v4). That makes this a good time to introduce a minimum Node.js version, and to get feedback on what we ought to support.Generally at Architect, we try to align our releases with Node.js Maintenance LTS + Active LTS releases, but there are numerous exceptions and plenty of reasons to allow support of certain EOL versions.
In this case, I'd propose we set Dynalite's minimum Node.js version to v14 (which went EOL in April of this year), namely so we can employ Architect's lint config and begin authoring with more modern syntax.
Anyway, please do let us know what you think!
The text was updated successfully, but these errors were encountered: