Skip to content

Commit 3a50892

Browse files
committed
replace webpack hasErrors calls - bug creates long logs
1 parent 3c42b19 commit 3a50892

File tree

3 files changed

+7
-10
lines changed

3 files changed

+7
-10
lines changed

devtools/regl_codegen/server.js

+2-3
Original file line numberDiff line numberDiff line change
@@ -97,9 +97,8 @@ compiler = webpack(devtoolsConfig);
9797
compiler.run(function(devtoolsErr, devtoolsStats) {
9898
if(devtoolsErr) {
9999
console.log('err:', devtoolsErr);
100-
} else if(devtoolsStats.hasErrors()) {
101-
if(!devtoolsStats.errors) console.log(devtoolsStats);
102-
else console.log('stats.errors:', devtoolsStats.errors);
100+
} else if(devtoolsStats.errors && devtoolsStats.errors.length) {
101+
console.log('stats.errors:', devtoolsStats.errors);
103102
} else {
104103
console.log('success:', devtoolsConfig.output.path + '/' + devtoolsConfig.output.filename);
105104

devtools/test_dashboard/server.js

+4-6
Original file line numberDiff line numberDiff line change
@@ -71,9 +71,8 @@ compiler = webpack(devtoolsConfig);
7171
compiler.run(function(devtoolsErr, devtoolsStats) {
7272
if(devtoolsErr) {
7373
console.log('err:', devtoolsErr);
74-
} else if(devtoolsStats.hasErrors()) {
75-
if(!devtoolsStats.errors) console.log(devtoolsStats);
76-
else console.log('stats.errors:', devtoolsStats.errors);
74+
} else if(devtoolsStats.errors && devtoolsStats.errors.length) {
75+
console.log('stats.errors:', devtoolsStats.errors);
7776
} else {
7877
console.log('success:', devtoolsConfig.output.path + '/' + devtoolsConfig.output.filename);
7978
}
@@ -84,9 +83,8 @@ compiler.run(function(devtoolsErr, devtoolsStats) {
8483
compiler.watch({}, function(err, stats) {
8584
if(err) {
8685
console.log('err:', err);
87-
} else if(stats.hasErrors()) {
88-
if(!stats.errors) console.log(stats);
89-
else console.log('stats.errors:', stats.errors);
86+
} else if(stats.errors && stats.errors.length) {
87+
console.log('stats.errors:', stats.errors);
9088
} else {
9189
console.log('success:', config.output.path + '/' + config.output.filename);
9290
}

tasks/util/browserify_wrapper.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ module.exports = function _bundle(pathToIndex, pathToBundle, opts, cb) {
5454
compiler.run(function(err, stats) {
5555
if(err) {
5656
console.log('err:', err);
57-
} if(stats.hasErrors()) {
57+
} if(stats.errors && stats.errors.length) {
5858
console.log('stats.errors:', stats.errors);
5959
} else {
6060
console.log('success:', config.output.path + '/' + config.output.filename);

0 commit comments

Comments
 (0)