Skip to content
This repository was archived by the owner on May 1, 2020. It is now read-only.

Commit 49c0afb

Browse files
committed
fix(uglifyjs): better error msg reporting
1 parent d9d000a commit 49c0afb

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

src/uglifyjs.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import * as uglify from 'uglify-js';
55

66
import { Logger } from './logger/logger';
77
import { fillConfigDefaults, generateContext, getUserConfigFile } from './util/config';
8+
import { BuildError } from './util/errors';
89
import { writeFileAsync } from './util/helpers';
910
import { BuildContext, TaskInfo } from './util/interfaces';
1011
import { runWorker } from './worker-client';
@@ -20,8 +21,8 @@ export function uglifyjs(context: BuildContext, configFile?: string) {
2021
.then(() => {
2122
logger.finish();
2223
})
23-
.catch(err => {
24-
throw logger.fail(err);
24+
.catch((err: BuildError) => {
25+
throw logger.fail(new BuildError(err));
2526
});
2627
}
2728

@@ -53,6 +54,10 @@ export function uglifyjsWorkerImpl(context: BuildContext, uglifyJsConfig: Uglify
5354
}
5455

5556
return Promise.all(promises);
57+
}).catch((err: any) => {
58+
// uglify has it's own strange error format
59+
const errorString = `${err.message} in ${err.filename} at line ${err.line}, col ${err.col}, pos ${err.pos}`;
60+
throw new BuildError(new Error(errorString));
5661
});
5762
}
5863

0 commit comments

Comments
 (0)