search

cssinjs / aphrodite-jss

Aphrodite-like API on top of JSS.
MIT License
92 stars 17 forks source link

Outdated node engine specified in package.json #9

Closed cryptoquick closed 7 years ago

cryptoquick commented 7 years ago

Is there a reason the package.json specifies that this package is incompatible with Node 7? This outright breaks a yarn install.

spadin commented 7 years ago

Have the same issue. I used yarn add aphrodite-jss --ignore-engines and it works, but it's annoying because I have to use --ignore-engines for any other yarn install going forward as well. Made a PR to remove this requirement.

kof commented 7 years ago

I think we can do the same we did for the jss core https://github.com/cssinjs/jss/blob/master/package.json#L27 because node is for devDependencies only.

kof commented 7 years ago

merged, published v1.0.3