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

Use Object.keys as Object.values is not supported in node<7 #295

Merged
merged 1 commit into from Oct 9, 2020

Conversation

klapec
Copy link
Contributor

@klapec klapec commented Oct 4, 2020

Object.values was introduced in 138b101 and it is not supported in node.js until version 7, so it's a mismatch with what's declared in the engines section of package.json (node >= 6).

This fixes #294

Object.values was introduced in 138b101
and it is not supported in node.js until version 7, so it's a mismatch
with what's declared in the `engines` section of package.json (node >= 6).
@coveralls
Copy link

Coverage Status

Coverage increased (+0.06%) to 91.195% when pulling 7385bc4 on klapec:fix-object-values into eefc98a on fb55:master.

@macmilan
Copy link

macmilan commented Oct 7, 2020

@here any update?

@klapec
Copy link
Contributor Author

klapec commented Oct 7, 2020

@macmilan So far I had to pin the dependency to a last-working version ("css-what": "3.3.0") but it'd be lovely to have this fixed.

@csergiu
Copy link

csergiu commented Oct 7, 2020

@rails/webpacker uses this in its' dependency tree and we can't deploy because of it. Any way we can force it to use 3.3.0 ?

@fb55 fb55 merged commit 98a0489 into fb55:master Oct 9, 2020
@klapec klapec deleted the fix-object-values branch October 9, 2020 17:36
@fb55
Copy link
Owner

fb55 commented Oct 9, 2020

Thanks a lot @klapec! I've added eslint-plugin-node in de90133 to prevent similar issues from happening again and published a new version, v3.4.2.

@klapec
Copy link
Contributor Author

klapec commented Oct 9, 2020

Awesome, thanks for merging @fb55! 🙂

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.

Broken on node.js < 7 due to Object.values usage
5 participants