Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Node 4] Update .travis.yml to install Node 4.x #2548

Merged
merged 1 commit into from
Sep 16, 2015

Conversation

ide
Copy link
Contributor

@ide ide commented Sep 4, 2015

The automated tests will install Node 4.x with nvm before running.

Test Plan: See if Travis tests pass.

@ide ide mentioned this pull request Sep 4, 2015
5 tasks
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 4, 2015
@ide ide force-pushed the node-convergence-tests branch 2 times, most recently from 44e5506 to b5efad9 Compare September 8, 2015 19:57
@cpojer
Copy link
Contributor

cpojer commented Sep 15, 2015

Can you remove the change to package.json? I'll update jest to 0.5.5 and then I can merge your PR.

Jest 0.5.2 ships with a fix for bound functions, which RN exercises. And with this Travis change, the automated tests will install Node 4.x with nvm before running.

Test Plan:`nvm use 4 && npm i && npm test`
See if Travis tests pass.
@ide ide force-pushed the node-convergence-tests branch from b5efad9 to fb59425 Compare September 15, 2015 20:59
@ide
Copy link
Contributor Author

ide commented Sep 15, 2015

@cpojer -- just the travis change now.

cpojer added a commit that referenced this pull request Sep 16, 2015
[Node 4] Update .travis.yml to install Node 4.x
@cpojer cpojer merged commit e056c62 into facebook:master Sep 16, 2015
@ide ide deleted the node-convergence-tests branch September 16, 2015 21:20
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants