Open timkelty opened 6 years ago
For the most part, this looks good.
Do we need to rename options.loader
to options.extract
in this PR? Seems like we would be able to add support for hot loading without making breaking changes.
If you have a minute, could you rebase these changes off of master (which now includes changes from #2)?
Also... @timkelty appreciate your work here 🙌
@jaridmargolin don't need to for this PR, no. Just included it since we have multiple "loaders" now.
However, the currently released version also has different option names (loaderOptions, pluginOptions)
Ahh I see now. Makes complete sense. Leave it in and I will publish major version.
Rebasing in a sec
@jaridmargolin ok, should be ready
Adding css-hot-loader support
This also clarifies some options:
options.cssHotLoader
options.loader
tooptions.extract
[hash]
so that HMR can workNote: I branched this off of #2 to avoid conflicts, as it seemed it would be merged soon.