Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

Update to Webpack 4 #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

argyleink
Copy link

since the starter is for css only I used that new webpack feature that lets css files be entry points. it doesnt really do much other than delete a file from the repo, and maybe that js entry file css loading strategy was better than this one?

i'm not married to this pr much, doesn't look like there's much to gain. but there's some work in here that could save ya time. 👍

@argyleink
Copy link
Author

oh snap, this one is good too #9

@adamwathan adamwathan mentioned this pull request Mar 19, 2018
@adamwathan
Copy link
Member

Nice! Feel free to chime in on discussion I just started there; trying to figure out if it makes more sense for us to do an HTML or CSS entry point for this starter project.

@argyleink
Copy link
Author

Wp4, so new, its hard to have an opinion. Is the goal of this repo to give someone a webpack app starter or tailwind sample? Are users visiting this repo to start net new or integrate into existing setups?

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

Successfully merging this pull request may close these issues.

None yet

2 participants