Skip to content

Add .node-version file #272

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

Merged
merged 1 commit into from
Mar 27, 2019
Merged

Conversation

kemuridama
Copy link
Contributor

@kemuridama kemuridama commented Mar 17, 2019

Describe in detail the problem you had and how this PR fixes it

Many developers are using **env tools such as nodenv to treat various version environments in each project.
nodenv is specifying node version for projects using by .node-version file.
So, this PR adds .node-version file for using nodenv and sets node version v8.15.0 according to Dockerfile.

Is there an open issue you can link to?

None.

@kemuridama kemuridama requested a review from kylecarbs as a code owner March 17, 2019 12:13
@kylecarbs
Copy link
Member

Do lots of people want this addition? It's very small, but I could see this getting out of hand with version files.

@kylecarbs kylecarbs added the waiting-for-info Waiting for more information from submitter label Mar 21, 2019
@kemuridama
Copy link
Contributor Author

Maybe yes. Some contributors or developers who try to build code-server locally fail to build code-server due to Node.js version does not match its version requiring to build code-server. Because
latest Node.js version is v11, they are often used it mainly and switch other version using nodenv.

@kylecarbs kylecarbs merged commit c7ae12c into coder:master Mar 27, 2019
@kemuridama kemuridama deleted the add_node_version branch March 28, 2019 02:17
@MichaelDesantis MichaelDesantis mentioned this pull request Apr 9, 2019
3 tasks
code-asher pushed a commit that referenced this pull request Jun 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting-for-info Waiting for more information from submitter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants