Switch to our forked version of less-plugin-autoprefix.

Review Request #11366 — Created Jan. 11, 2021 and submitted — Latest diff uploaded

Information

Djblets
release-2.x

Reviewers

Official versions of less-plugin-autoprefix don't work with modern
versions of NodeJS, due to a regression in NodeJS that was only
partially fixed in recent releases.

While waiting for upstream to accept a patch for this, we've gone ahead
and forked it with our fix. This change switches us to installing and
using it.

The less-imports.js had to be modified to be less restrictive when it
comes to parameters. It now just ignores everything it doesn't
understand.

This change will require re-running either setup.py develop or
setup.py install_node_modules.

Built a Djblets package successfully. Verified the forked plugin ran.

Diff Revision 1 (Latest)

Commits

First Last Summary ID Author
Switch to our forked version of less-plugin-autoprefix.
Official versions of `less-plugin-autoprefix` don't work with modern versions of NodeJS, due to a regression in NodeJS that was only partially fixed in recent releases. While waiting for upstream to accept a patch for this, we've gone ahead and forked it with our fix. This change switches us to installing and using it. The `less-imports.js` had to be modified to be less restrictive when it comes to parameters. It now just ignores everything it doesn't understand. This change will require re-running either `setup.py develop` or `setup.py install_node_modules`.
6a9b48f567ed87d66090c00fa5359100c341d359 Christian Hammond
djblets/dependencies.py
djblets/settings.py
djblets/pipeline/compilers/less-imports.js
Loading...