Skip to content

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

Merged
fb55 merged 1 commit intofb55:masterfrom
klapec:fix-object-values
Oct 9, 2020
Merged

Use Object.keys as Object.values is not supported in node<7#295
fb55 merged 1 commit intofb55:masterfrom
klapec:fix-object-values

Conversation

@klapec
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown

macmilan commented Oct 7, 2020

@here any update?

@klapec
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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