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

node-v3.0.0 #3650

Merged
merged 1 commit into from
Jan 21, 2016
Merged

node-v3.0.0 #3650

merged 1 commit into from
Jan 21, 2016

Conversation

mikemorris
Copy link
Contributor

Document dropping support for Node.js v0.10.x, add engines field to package.json and update changelog.

@mikemorris mikemorris added the Node.js node-mapbox-gl-native label Jan 21, 2016
@mikemorris mikemorris merged commit 0a2e4af into master Jan 21, 2016
@mikemorris mikemorris deleted the node-v3.0.0 branch January 21, 2016 17:40
@@ -31,6 +31,9 @@
"request": "^2.67.0",
"tape": "^4.2.2"
},
"engines": {
"node": ">=4.2.1"

Choose a reason for hiding this comment

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

@mikemorris Do versions before 4.2.1 actually not work?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@bsudekum v4.2.0 introduced a bug that was fixed in v4.2.1 with nodejs/node@b3cbd13, and IIRC there was another bug in early v4.0.x releases. I could track down exactly which versions are incompatible, but this was simpler.

Choose a reason for hiding this comment

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

👍 just curious

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Node.js node-mapbox-gl-native
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants