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

add Node 5 to devEngines #5353

Merged
merged 1 commit into from
Nov 4, 2015
Merged

add Node 5 to devEngines #5353

merged 1 commit into from
Nov 4, 2015

Conversation

chicoxyzzy
Copy link
Contributor

No description provided.

@chicoxyzzy
Copy link
Contributor Author

I've tested React on Node.js 5 working on #5352 and #5390 and all seems to be 👌

@jimfb
Copy link
Contributor

jimfb commented Nov 4, 2015

Yeah, I've run unit tests with v5, haven't seen any issues. I'll leave it open for a day in case @zpao or others want to raise a flag, otherwise, let's take it.

@jimfb jimfb self-assigned this Nov 4, 2015
@zpao
Copy link
Member

zpao commented Nov 4, 2015

Yea, alright. But same goes as for the npm 3 bit. We don't use it so if we run into issues, we'll drop it back down to LTS.

zpao added a commit that referenced this pull request Nov 4, 2015
@zpao zpao merged commit 80af7c7 into facebook:master Nov 4, 2015
@chicoxyzzy chicoxyzzy deleted the patch-1 branch November 4, 2015 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants