Seravo/wordpress

NodeJS upgrade issues, frequently with node-sass

ottok opened this issue · 6 comments

ottok commented

Note directly related to our project template, but still a common issue worth documenting.

Many have reported problems with node-sass and the latest NodeJS version. Root cause and its solution seem to be dlmanning/gulp-sass#748

Another issue seems to be stray caches. To ensure that npm or yarn installs the latest versions of everything, these tips might help:

rm -rf node_modules .cache-loader package-lock.json yarn.lock
npm cache clean -f
npm install

These might also sometimes help:

npm --depth 9999 update
npm rebuild

Note, that you might also get different results with npm install X and yarn add X in some corner cases, even though they in principle obey the same package.json

ottok commented

Naturally, if the package.json is fixed to some very old NodeJS libraries that don't run on NodeJS 12, then those dependencies need to be bumped.

As the package.json file is provided in the repository (and rightfully so), would it make sense to include a .nvmrc file (or some equivalent) as well? That would make it easier to get to a known working state with the site and get people started building stuff faster.

ottok commented

@Kimitri Would you like to make a suggestion in the form of a pull request?

@ottok Sure! That would be nice. Do you guys have any contribution guidelines that I should check out first?

ottok commented

@ottok Sure! That would be nice. Do you guys have any contribution guidelines that I should check out first?

Nothing special. Just open a PR like on any other Github project. If the solution is not a piece of code/automation, you can also make a documentation PR agains https://github.com/Seravo/docs (that is https://seravo.com/docs/).

ottok commented

Related: Occasionally people reported issues with gulp-sass wanting to build, and that was because the version we used was so old that it wasn't available as a binary from upstream anymore. Fixed via #108