Skip to content
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

Document and warn about NestJs 7 compatible node versions #4870

Closed
elAndyG opened this issue Jun 2, 2020 · 3 comments
Closed

Document and warn about NestJs 7 compatible node versions #4870

elAndyG opened this issue Jun 2, 2020 · 3 comments

Comments

@elAndyG
Copy link

elAndyG commented Jun 2, 2020

Feature Request

Is your feature request related to a problem? Please describe.

When following the documented nest new project-name, it does not work with Node version 12/13 based on the following issue. Unfortunately, the latest LTS version of node is is 12 so not all developers can switch to 14.

Google Cloud App Engine is only compatible with the LTS version.

Relates to nestjs/nest-cli#724

Describe the solution you'd like

  1. Document that NestJS 7 is compatible with 14.
  2. Document that if you want to use the LTS version of node, use the CLI version 6.
  3. if a developer has version 7 installed, warn them in the console that they need to use version 14 before you start scaffolding a solution.

Teachability, Documentation, Adoption, Migration Strategy

The ideal solution is to allow NestJS to work with Node 12 LTS.

What is the motivation / use case for changing the behavior?

I spent quite a few hours researching why a fresh nest new project-name couldn't simply start up using npm run start when my older versions of nest were working just fine. I'd also like to be able to use the latest version of Nest if i could, but if i read documentation stating that Node LTS is only compatible with Nest 6, i would have happily followed those directions.

@elAndyG elAndyG added needs triage This issue has not been looked into type: enhancement 🐺 labels Jun 2, 2020
@kamilmysliwiec kamilmysliwiec added needs clarification and removed needs triage This issue has not been looked into type: enhancement 🐺 labels Jun 3, 2020
@kamilmysliwiec
Copy link
Member

It does not work with Node version 12/13 based on the following issue. Unfortunately, the latest LTS version of node is is 12 so not all developers can switch to 14.

I'm personally using Node v12 (LTS) and I don't have any issues with the nest CLI. What Node.js version do you use specifically?

@elAndyG
Copy link
Author

elAndyG commented Jun 3, 2020

@kamilmysliwiec
On my macOS, i have the following:
nest: 7.2.0
npm: 6.13.1
node: 13.2.0

My teammate's WindowsOS has the following:
nest: 7.2.0
npm: 6.12.1
node: 13.1.0

Both of us are having the same problem as nestjs/nest-cli#724. I'll try downgrading to the latest 12 and see if that works. Maybe 13 is the root of my problems. If thats the case, i'll update the ticket to state "13" and remove 12 from the information.

@kamilmysliwiec
Copy link
Member

I would not suggest using Node 13 since, as you can see, it's quite buggy. Use either LTS 12 or the latest version 14. Nest (and other packages) should work properly on both.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants