Skip to content
This repository was archived by the owner on Dec 4, 2017. It is now read-only.

chore(node): recommend v5 via .nvmrc #1623

Closed
wants to merge 1 commit into from

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Jun 8, 2016

@wardbell @Foxandxss @kwalrath : enough of us have had enough problems with node that I think it is worth while setting the .nvmrc to v5.

@Foxandxss
Copy link
Member

Well, for ourselves at this project yes. But we are a few that use NVM, the rest doesn't.

@chalin
Copy link
Contributor Author

chalin commented Jun 8, 2016

Sure, but this addition won't hurt those who don't use nvm.

@Foxandxss
Copy link
Member

That is right.

I don't like to constrain people to use certain versions and only recommend.

But I don't really mind adding this.

@chalin
Copy link
Contributor Author

chalin commented Jun 8, 2016

As you know it doesn't force anyone to use v5 (one can always nvm use 6);
its just that v5 is the best default for now.

On Wed, Jun 8, 2016 at 12:01 PM, Jesús Rodríguez notifications@github.com
wrote:

That is right.

I don't like to constrain people to use certain versions and only
recommend.

But I don't really mind adding this.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#1623 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AD8u-ftqVvdGCQIeVu0tGKk5SUfsGzEKks5qJxGGgaJpZM4IxK1b
.

@chalin chalin force-pushed the chalin-chore-nvmrc branch from fa7a4f7 to cd66772 Compare June 13, 2016 14:14
@wardbell wardbell closed this in 4dab164 Jun 13, 2016
@chalin chalin deleted the chalin-chore-nvmrc branch June 13, 2016 20:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants