-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
Decide on what to do with Node version in dockerization guide. #2988
Comments
The current text uses Node.js 10 and that’s still in support until next year (April 2021). Doing nothing for now and leaving the text as-is is an option. |
The text explicitly recommends the latest LTS version (12), while the text inconsequently mentions versions 8 and 10. Please see the PRs. |
I'd be ok with leaving the doc as is, but then adding a line somewhere that says something like this: "we recommend the latest lts version which was 10 at the time this guide was written. Please use the latest lts version instead of 10 as shown in the examples" |
Closing as it seems the parties are fine with the current content. And the guide was updated. |
Currently, an old version of LTS Node is mentioned in the guide. A fix has been proposed in PR #2953, and shortly after, #2983 followed. I'm opening this issue to avoid splitting the discussion between these two PRs.
The former lets the maintainers forget about manual version bumping which may be a source of inaccuracies (as proven in PR description), but may cause problems to the users, since the version is variable and changes with new Node releases.
The latter is safer for the users, but may again cause inaccuracies, also requires manual version bump with every LTS release. It also requires to add another point to an already long list in the release process.
I would personally go with the former. Only advantage of #2983 over #2953 is that it might protect the user from some discrepancies in case of a new Node LTS release. However I believe that problems emering from such discrepancies are very improbable. Additionally, even the best guide will not prevent from misuse of software.
The text was updated successfully, but these errors were encountered: