Skip to content
This repository has been archived by the owner on Feb 24, 2024. It is now read-only.

Adding clean-obsolete-chunks which will clean up on recompile #1013

Merged
merged 4 commits into from
Apr 10, 2018

Conversation

paganotoni
Copy link
Member

@paganotoni paganotoni commented Apr 7, 2018

This PR makes our webpack config clean obsolete chunks so instead of just cleaning all the public/assets folder this will remove only obsolete chunks, this will cause 2 great outcomes:

  • We no longer will have trash chunks after recompiling
  • Styles will load fast after we've already compiled (p.e application restart doesn't need to do the compilation from scratch).

Appart from that change I've also:

  • Added a repository field to generated package.json to avoid a warning that was being shown.
  • Split webpack related packages and UI related packages in the package.json by using dependencies and devDependencies respectively.

@paganotoni paganotoni requested a review from markbates April 7, 2018 22:58
@paganotoni
Copy link
Member Author

Fixes #1012 cc @markbates

@paganotoni paganotoni added this to the 0.11.1 milestone Apr 9, 2018
@paganotoni
Copy link
Member Author

@markbates You ok if I merge this one? I've tested it and is looking good.

@markbates markbates merged commit 7cb26b6 into development Apr 10, 2018
@markbates markbates deleted the task/moving-to-clean-obsolete-chunks branch April 10, 2018 15:15
@markbates
Copy link
Member

@paganotoni sorry it took me so long to get to it.

@paganotoni
Copy link
Member Author

no worries! thanks!

markbates pushed a commit that referenced this pull request Apr 10, 2018
* adding clean-obsolete-chunks which will clean up on recompile

* adding dot between css name and hash
stanislas-m pushed a commit that referenced this pull request May 12, 2018
* adding clean-obsolete-chunks which will clean up on recompile

* adding dot between css name and hash
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.

2 participants