Skip to content

[DRAFT MAJOR WIP] Add Node.js engines field which drops old Node.js versions#443

Draft
brody2consult wants to merge 1 commit into
mhevery:masterfrom
brody2consult:node-engines
Draft

[DRAFT MAJOR WIP] Add Node.js engines field which drops old Node.js versions#443
brody2consult wants to merge 1 commit into
mhevery:masterfrom
brody2consult:node-engines

Conversation

@brody2consult

Copy link
Copy Markdown
Collaborator

This should be in an another new major release as tracked in #436, drops support for Node.js pre-8.0

@brody2consult brody2consult left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

TODO items

Comment thread package.json
"author": "Misko Hevery <misko@hevery.com>",
"license": "MIT",
"engines": {
"node": ">= 8.0.0"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
"node": ">= 8.0.0"
"node": ">= 10"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

and use npm:please-upgrade-node to enforce minimum Node.js requirement more strongly (#444)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant