• 
      

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

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

    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.

    Summary ID
    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
    david
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-2.x (ea792d2)