Skip to content
This repository has been archived by the owner on Feb 6, 2023. It is now read-only.

Add Node.js version 10 for CI #1909

Closed
wants to merge 3 commits into from
Closed

Add Node.js version 10 for CI #1909

wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Oct 21, 2018

Summary

This PR adds nodejs version 10 for the Travis Ci build

Details:

image

Test Plan

Travis CI test

@ghost ghost changed the title Add node version 10 for ci Add Node.js version 10 for CI Oct 21, 2018
@ghost
Copy link
Author

ghost commented Oct 21, 2018

I updated natives in yarn.lock to 1.1.6 (^1.1.3) to fix this issue gulpjs/gulp#2162.
Alternatively,gulp can be updated to version 4.

@claudiopro
Copy link
Contributor

Thanks for contributing a fix for the Gulp crash on Node 10, @noelyoo! I'd prefer to upgrade natives instead of gulp as that change might have a broader blast radius.

May I ask you to please use Yarn's selective dependency version resolution instead of editing the lockfile directly? I tested this addition to package.json and it works:

  "resolutions": {
    "gulp/**/natives": "1.1.3"
  },

@claudiopro claudiopro self-requested a review October 21, 2018 19:38
Copy link
Contributor

@claudiopro claudiopro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please take a look at my comments! Otherwise, looking good 👍

package.json Show resolved Hide resolved
@claudiopro claudiopro self-assigned this Oct 22, 2018
Copy link
Contributor

@claudiopro claudiopro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing this improvement @noelyoo 👍

Copy link

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

claudiopro is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@ghost
Copy link
Author

ghost commented Oct 22, 2018

@claudiopro I’m happy to contribute 👍

jdecked pushed a commit to twitter-forks/draft-js that referenced this pull request Oct 9, 2019
Summary:
**Summary**

This PR adds nodejs version `10` for the Travis Ci build

Details:
* https://github.com/nodejs/Release#end-of-life-releases
* https://docs.travis-ci.com/user/languages/javascript-with-nodejs/#specifying-nodejs-versions

![image](https://user-images.githubusercontent.com/25740248/47252831-f1f67580-d485-11e8-8ae0-1525cfd154df.png)

**Test Plan**

Travis CI test
Pull Request resolved: facebookarchive#1909

Differential Revision: D10487838

fbshipit-source-id: 98d98ea38da66752c6c73fd379acb024058f09a3
alicayan008 pushed a commit to alicayan008/draft-js that referenced this pull request Jul 4, 2023
Summary:
**Summary**

This PR adds nodejs version `10` for the Travis Ci build

Details:
* https://github.com/nodejs/Release#end-of-life-releases
* https://docs.travis-ci.com/user/languages/javascript-with-nodejs/#specifying-nodejs-versions

![image](https://user-images.githubusercontent.com/25740248/47252831-f1f67580-d485-11e8-8ae0-1525cfd154df.png)

**Test Plan**

Travis CI test
Pull Request resolved: facebookarchive/draft-js#1909

Differential Revision: D10487838

fbshipit-source-id: 98d98ea38da66752c6c73fd379acb024058f09a3
aforismesen added a commit to aforismesen/draft-js that referenced this pull request Jul 12, 2024
Summary:
**Summary**

This PR adds nodejs version `10` for the Travis Ci build

Details:
* https://github.com/nodejs/Release#end-of-life-releases
* https://docs.travis-ci.com/user/languages/javascript-with-nodejs/#specifying-nodejs-versions

![image](https://user-images.githubusercontent.com/25740248/47252831-f1f67580-d485-11e8-8ae0-1525cfd154df.png)

**Test Plan**

Travis CI test
Pull Request resolved: facebookarchive/draft-js#1909

Differential Revision: D10487838

fbshipit-source-id: 98d98ea38da66752c6c73fd379acb024058f09a3
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.

2 participants