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

[npm] Update to stacktrace-parser@0.1.3 and ws@0.8.0 to satisfy io.js 3 / Node 4 #2398

Merged
merged 1 commit into from
Sep 2, 2015

Conversation

ide
Copy link
Contributor

@ide ide commented Aug 21, 2015

The previous version of stacktrace-parser 0.1.2 had an line in its package.json file that required Node <= 2.x. That line was removed, so it no longer warns on io.js 3.x or Node 4.x.

Similarly, ws 0.8.0 was published with support for the new V8, so it compiles with io.js 3.x and therefore should work with Node 4.x.

Updated the Travis file as well to run on io.js 3.x.

Fixes #2258, #2455

Test Plan:

$ nvm use iojs-v3
Now using io.js v3.1.0 (npm v2.13.3)

$ npm i
<no compiler errors>

@facebook-github-bot facebook-github-bot added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Aug 21, 2015
@ide ide force-pushed the iojs-3-deps branch 5 times, most recently from ab1cbff to c8b06f7 Compare August 26, 2015 23:04
@brentvatne
Copy link
Collaborator

👍 👍

@ide
Copy link
Contributor Author

ide commented Aug 30, 2015

Node 4.0 is expected to ship next week Sep. 3 or Sep. 7, so it would be timely to get this in soon.

… 3 / Node 4

The previous version of stacktrace-parser 0.1.2 had an line in its package.json file that required Node <= 2.x. That line was removed, so it no longer warns on io.js 3.x or Node 4.x.

Similarly, ws 0.8.0 was published with support for the new V8, so it compiles with io.js 3.x and therefore should work with Node 4.x.

Updated the Travis file as well to run on io.js 3.x.

Test Plan:
```
$ nvm use iojs-v3
Now using io.js v3.1.0 (npm v2.13.3)

$ npm i
<no compiler errors>
```
@ide
Copy link
Contributor Author

ide commented Sep 1, 2015

Updated this to use pinned versions instead of the npm caret^

cc @tadeuzagallo

@tadeuzagallo
Copy link
Contributor

@facebook-github-bot import

tadeuzagallo added a commit that referenced this pull request Sep 2, 2015
[npm] Update to stacktrace-parser@0.1.3 and ws@0.8.0 to satisfy io.js 3 / Node 4
@tadeuzagallo tadeuzagallo merged commit b92efd1 into facebook:master Sep 2, 2015
sahrens pushed a commit to sahrens/react-native that referenced this pull request Sep 3, 2015
… 3 / Node 4

Summary:
The previous version of stacktrace-parser 0.1.2 had an line in its package.json file that required Node <= 2.x. That line was removed, so it no longer warns on io.js 3.x or Node 4.x.

Similarly, ws 0.8.0 was published with support for the new V8, so it compiles with io.js 3.x and therefore should work with Node 4.x.

Updated the Travis file as well to run on io.js 3.x.

Fixes facebook#2258, facebook#2455

Closes facebook#2398
Github Author: James Ide <ide@jameside.com>
@ide ide deleted the iojs-3-deps branch September 4, 2015 11:38
ide added a commit that referenced this pull request Sep 8, 2015
… 3 / Node 4

Summary:
The previous version of stacktrace-parser 0.1.2 had an line in its package.json file that required Node <= 2.x. That line was removed, so it no longer warns on io.js 3.x or Node 4.x.

Similarly, ws 0.8.0 was published with support for the new V8, so it compiles with io.js 3.x and therefore should work with Node 4.x.

Updated the Travis file as well to run on io.js 3.x.

Fixes #2258, #2455

Closes #2398
Github Author: James Ide <ide@jameside.com>
# 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.

5 participants