-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
docs: local setup notes #5974
docs: local setup notes #5974
Conversation
This is a Node.js project and you need to have Node.js installed on your machine. You can download it from [here](https://nodejs.org/). We test versions 14 and 16 of Node in the CI, so it's recommended to use one of these versions, or the latest of: 16.20.2 | ||
|
||
```bash | ||
nvm use 16.20.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RE discussion, we should use this version locally to ensure consistent machine to CI.
16.20.2
comes from the latest 16 in https://nodejs.org/en/about/previous-releases and satisfies the node-version: [14.17, 16.x]
matrix in the CI test runner.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙇♂️
Based on the releases, we should probably switch to [20.x, 22.x]
soon.
Nice addition @danstarns Thank you |
This PR adds notes about setting up the repo locally and making some assumptions about the dev flow of other contributors.
Related discussion: