Skip to content

Display JS and CSS bundle sizes after build #229

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.

Sign up for GitHub

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

Merged
gaearon merged 3 commits into from
Jul 27, 2016
Merged

Display JS and CSS bundle sizes after build #229

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e2f4aa9
Display JS and CSS bundle sizes after build
lvwrence Jul 27, 2016
52a8605
Print gzipped filesize, address nits
lvwrence Jul 27, 2016
598a832
Use filesize for human readable output.
lvwrence Jul 27, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,9 @@
"eslint-plugin-react": "5.2.2",
"extract-text-webpack-plugin": "1.0.1",
"file-loader": "0.9.0",
"filesize": "^3.3.0",
"fs-extra": "0.30.0",
"gzip-size": "^3.0.0",
"html-webpack-plugin": "2.22.0",
"json-loader": "0.5.4",
"opn": "4.0.2",
Expand Down
16 changes: 16 additions & 0 deletions scripts/build.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@

process.env.NODE_ENV = 'production';

var fs = require('fs');
var filesize = require('filesize');
Copy link
Copy Markdown

@taion taion Jul 27, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You want gzip-size, actually.

Copy link
Copy Markdown
Contributor Author

@lvwrence lvwrence Jul 27, 2016 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was under the impression that the assets have already been gzipped, so I just needed to print out their filesizes? If that's not the case, does this mean we have to read every file to get their estimated gzipped size?

Copy link
Copy Markdown
Contributor Author

@lvwrence lvwrence Jul 27, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I think I get it now. Gonna get some food and come back to this.

Copy link
Copy Markdown
Contributor Author

@lvwrence lvwrence Jul 27, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using gzip-size now.

Copy link
Copy Markdown
Contributor Author

@lvwrence lvwrence Jul 27, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also we probably still want filesize for human readable file sizes.

var gzipSize = require('gzip-size');
var rimrafSync = require('rimraf').sync;
var webpack = require('webpack');
var config = require('../config/webpack.config.prod');
Expand All @@ -18,6 +21,16 @@ var paths = require('../config/paths');
// if you're in it, you don't end up in Trash
rimrafSync(paths.appBuild + '/*');

function logBuildSize(assets, extension) {
for (var i = 0; i < assets.length; i++) {
var asset = assets[i];
if (asset.name.endsWith('.' + extension)) {
var fileContents = fs.readFileSync(paths.appBuild + '/' + asset.name);
console.log('Size (gzipped) of ' + asset.name + ': ' + filesize(gzipSize.sync(fileContents)));
}
}
}

webpack(config).run(function(err, stats) {
if (err) {
console.error('Failed to create a production build. Reason:');
Expand Down Expand Up @@ -48,6 +61,9 @@ webpack(config).run(function(err, stats) {
console.log(' hs');
console.log(' ' + openCommand + ' http://localhost:8080');
console.log();
var assets = stats.toJson()['assets'];
logBuildSize(assets, 'js');
logBuildSize(assets, 'css');
}
console.log('The bundle is optimized and ready to be deployed to production.');
});