Effectively make stable an alias for lts. (Solves issue #335) #467
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request Template:
Describe what you did
Made stable an alias for lts.
How you did it
Removed stable from documented options, other than as an alias for lts.
Modified display_latest_stable_version to return LTS version.
Retained existing stable code for backwards compatibility.
How to verify it doesn't effect the functionality of n
n --latest: 8.8.1
n --lts: 6.11.5
n --stable: 6.11.5
n latest && node --version: v8.8.1
n stable && node --version: v6.11.5
n latest && node --version: v8.8.1
n lts && node --version: v6.11.5
If this solves an issue, please put issue in PR notes.
Solves issue #335 where stable was using old pattern of latest even minor version number.
Using lts is a good match for "stable", but is a significant change to the erroneous behaviour which was sometimes same as latest .
If this solves an issue, please include the output of issue that had problems and then the fixed output from the same command.
Say node latest was at 8.9.2.
Old:
n --latest: 8.9.2
n --lts: 6.11.5
n --stable: 8.8.1
New:
n --latest: 8.9.2
n --lts: 6.11.5
n --stable: 6.11.5
Squash any unnecessary commits to keep history clean as possible
Place description for the changelog in PR so we can tally all changes for any future release
Change stable to be an alias for lts.