-
Notifications
You must be signed in to change notification settings - Fork 484
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
Support lts/*
in .nvmrc files
#137
Comments
Let's say you have Node 8 installed, should The former is possible, by traversing on every I think that neither are good options because there are breaking changes between LTS versions (I have seen projects that bumping Node versions to 10 was problematic.) and therefore its okay to only support the What do you think? |
I think explicitly stating in the documentation that I just made this issue because the docs stated that |
However, if you do want to support it, I believe the 2nd of the two options you listed, I am not familiar with the internals of
|
As @dcodeIO has mentioned on Twitter (or maybe I inferred it from that thread), a good case for
It can be! The only thing left to understand now is how to make the most performant latest lts checks.
|
I use lts/* on all my linting builds, as well as |
How can you setup to use |
Right now, use a specific lts version, like |
nvm supports both; double dash for the command line, the slash form is for config files. |
Currently,
lts/*
is considered an acceptable version bynvm
. It knows to treat this as the latest LTS version. However,fnm
doesn't seem to understand this syntax even though it's a valid .nvmrc config. Would it be possible to support this?The text was updated successfully, but these errors were encountered: