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

feat: better defaults (0CJS) with config generation #942

Merged
merged 3 commits into from
Jun 8, 2019
Merged

Conversation

evenstensberg
Copy link
Member

@evenstensberg evenstensberg commented Jun 7, 2019

What kind of change does this PR introduce?
Better default settings for init generator

Did you add tests for your changes?
N/A
If relevant, did you update the documentation?
N/A
Summary
Sets better defaults if user is in production and if the user is using defaults.

Does this PR introduce a breaking change?
N/A

Other information

After this is done, we can use the default config generated from webpack init --auto and use that as the default config for a webpack instance. After doing so, we can remove the isUsingDefaults logic from the generator (or most of it) and put it as a non-config requirement in the core cli.

TODO:

  • Prefetching
  • other things that might come up
  • More aggressive chunksplitting strategies on specialized builds

@evenstensberg evenstensberg changed the title feat: better defaults feat: better defaults (0CJS) with config generation Jun 7, 2019
@evenstensberg
Copy link
Member Author

We can make it offline support + PWA here as well. Will implement

@evenstensberg evenstensberg marked this pull request as ready for review June 8, 2019 14:38
@evenstensberg evenstensberg merged commit 858a532 into master Jun 8, 2019
@evenstensberg evenstensberg deleted the feat/0cjs branch June 8, 2019 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants