Provide support for lesscss plugins

[mfellner]

Since version 2 of less, Clean CSS is no longer bundled but has to be used as a plugin http://lesscss.org/usage/#command-line-usage-compress. The query-string configuration of less-loader is insufficient to pass plugin instances to less.

This change allows for plugins to be passed to less as specified here: http://lesscss.org/usage/#plugins-using-a-plugin-in-code.

[jhnns]

You're right. It's better to use the webpack options in this case (although it's usually discouraged). What do you think @sokra?

Could you namespace and rename the option key to lessPlugins since it's actual a config for plugins. Like this:

module.exports = {
    lessLoader: {
        lessPlugins: [...]
    }
};

We never used the term lesscss before and afaict it's only their URL, not their actual module name.

[sokra]

You're right. It's better to use the webpack options in this case (although it's usually discouraged). What do you think @sokra?

yes. If you want to be perfect allow to choose the key in the options by query parameter. This way you could avoid conflicts.

[jhnns]

If you want to be perfect allow to choose the key in the options by query parameter

That would be the perfect solution™ indeed 😀. But it should use a sane default like the npm name camelcased.

Something like:

require("!less-loader?config=someCustomName!./main.less")

module.exports = {
    someCustomName: {
        lessPlugins: [...]
    }
};

And config defaults to lessLoader.

[mfellner]

@jhnns, @sokra: I committed the suggested changes. Let me know if there are any other improvements I could make! 😄

[jhnns]

Nice! Good work 👍

Shipped with 2.2.0

[mfellner]

Thanks! Glad I could help.