Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
willhoney7 opened this issue Dec 30, 2016 · 5 comments
Open

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

willhoney7 opened this issue Dec 30, 2016 · 5 comments

Comments

@willhoney7
Copy link

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 andywer changed the title Discussion: Add html-loader to base config Discussion: Add html-loader to base config? Dec 30, 2016
@andywer
Copy link
Owner

andywer commented Dec 30, 2016

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
Copy link
Owner

andywer commented Dec 30, 2016

@willhoney7
Copy link
Author

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
Copy link
Owner

andywer commented Dec 30, 2016

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

@andywer
Copy link
Owner

andywer commented Jan 1, 2017

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 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants