Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Don't run on Node.js versions less than 8.6.0 #1418

Merged
merged 1 commit into from
Sep 18, 2019

Conversation

watson
Copy link
Member

@watson watson commented Sep 17, 2019

ESLint uses the spread operator which isn't supported in Node.js versions earlier than 8.6.0.

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[x] Bug fix
[ ] New feature
[ ] Other, please explain:

What changes did you make? (Give an overview)

Previously the cli would just check that the major version was 8 or above before continuing. This change now also looks at the minor version to ensure that the full version is v8.6.0 or above.

ESLint uses the spread operator which isn't supported in Node.js
versions earlier than 8.6.0.
@watson watson requested a review from feross September 17, 2019 19:41
Copy link
Member

@feross feross left a comment

Choose a reason for hiding this comment

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

LGTM!

@feross feross merged commit 7886564 into standard:master Sep 18, 2019
@feross
Copy link
Member

feross commented Sep 18, 2019

14.3.1

@watson watson deleted the min-node-version branch September 18, 2019 08:24
@watson
Copy link
Member Author

watson commented Sep 18, 2019

@feross Does the 14.3.1 comment mean that it will be released in v14.3.1, or that it has been released in that version? Because I don't see it on npm yet 🤔

I can publish the release if you like, but just haven't been paying attention to the convention lately 😅

@feross
Copy link
Member

feross commented Sep 18, 2019

Does the 14.3.1 comment mean that it will be released in v14.3.1, or that it has been released in that version?

It means that I released it on npm. Except in this case, I didn't actually do the release! The build was failing so I went to fix that and forgot to actually publish. Fixed now!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants