dependency requires node >= 8 about sleek HOT 3 CLOSED

Hey @benpatterson,

Is that a big issue? I think >= node 8 is the most recent stable version. Not sure from where puppeteer dependency comes from, perhaps it's a "dependency of a dependency"?

from sleek.

@janczizikow The EOL for node 6 is April 2019. (Node 4 EOL's this month.) It's your call if you want to take action...I just wanted to call this out since I ended up spending time troubleshooting an install failure; took some digging to determine why the failure was occurring when following the install steps. Could also be a doc update if you'd rather keep package.json where it is.

puppeteer is a dependency of penthouse. I'm happy to put in a PR if that would help, but it's all good if you want to handle this a different way. Thanks for taking the time to read through my issue.

from sleek.

Hey @benpatterson,

I just took a look: penthouse is a dependency of critical. It's a package to inline the critical CSS, which helps to improve page render time. Did you fix this issue on your machine by upgrading node or removing penthouse? I don't really want to give up on critical. Since this theme focuses on performance, having an autorunner task to inline the critical chunks of CSS it's rather a priority. I think the easiest solution would be to update the docs like you said.

Thanks to you for your time and creating this issue 👍

from sleek.