andywer / webpack-blocks

📦 Configure webpack using functional feature blocks.
MIT License
2.97k stars 94 forks source link

Discussion: Add html-loader to base config? #62

Open willhoney7 opened 7 years ago

willhoney7 commented 7 years ago

I think the html-loader is universal enough to add to the base config.

Thoughts?

https://github.com/andywer/webpack-blocks/blob/master/packages/webpack2/index.js

andywer commented 7 years ago

Right, this is a pretty common loader. But it also brings a bunch of additional transitive dependencies (see its package.json), thus taking a bit more time to npm install.

Guess I will just post a poll on twitter!

andywer commented 7 years ago

Here is the poll 😉 https://twitter.com/andywritescode/status/814873242833973248

willhoney7 commented 7 years ago

Awesome!

I could see it going either way. Since I usually use webpack for angular apps, the html-loader is a no brainer... Not sure if that need extends to react/vuejs development.

Perhaps it could become a new block? It could be dead simple. Just the html loader.

andywer commented 7 years ago

Sure. Let's consider the new block is plan B :)

andywer commented 7 years ago

Hmm. The poll suggests creating a block for it, but due to a really bad timing (NYE) participation was extremely bad.

I guess we should nevertheless just create new block. We could also create an Angular2 preset, for example :)