Skip to content
New issue

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

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

Already on GitHub? # to your account

[RFR] Optimize build process #477

Merged
merged 1 commit into from
Jun 1, 2015
Merged

[RFR] Optimize build process #477

merged 1 commit into from
Jun 1, 2015

Conversation

jpetitcolas
Copy link
Contributor

Add better uglification, optimize Webpack internal ids and remove eventually duplicated code.

Before:

$ ls -lh build/
total 2,8M
-rw-r--r-- 1 jpetitcolas jpetitcolas 1,4M May 29 10:01 ng-admin.min.css
-rw-r--r-- 1 jpetitcolas jpetitcolas 814K May 29 10:01 ng-admin.min.js
-rw-r--r-- 1 jpetitcolas jpetitcolas 7,0K May 29 10:01 ng-admin-only.min.css
-rw-r--r-- 1 jpetitcolas jpetitcolas 593K May 29 10:01 ng-admin-only.min.js

After:

$ ls -lh build/
total 2,7M
-rw-r--r-- 1 jpetitcolas jpetitcolas 1,4M May 29 10:02 ng-admin.min.css
-rw-r--r-- 1 jpetitcolas jpetitcolas 796K May 29 10:02 ng-admin.min.js
-rw-r--r-- 1 jpetitcolas jpetitcolas 7,0K May 29 10:02 ng-admin-only.min.css
-rw-r--r-- 1 jpetitcolas jpetitcolas 575K May 29 10:02 ng-admin-only.min.js

@manuquentin
Copy link
Contributor

Thanks for polar bears !

@jpetitcolas
Copy link
Contributor Author

image

@jeromemacias
Copy link
Contributor

Please do not commit built files to avoid conflict.

@jeromemacias
Copy link
Contributor

Can you rebase ?

fzaninotto added a commit that referenced this pull request Jun 1, 2015
@fzaninotto fzaninotto merged commit 82cee63 into master Jun 1, 2015
@fzaninotto fzaninotto deleted the build_optimizations branch June 1, 2015 19:01
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants