Skip to content
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

Log gulp error to Chart.js #5143

Merged
merged 13 commits into from
Jan 13, 2018
Merged

Log gulp error to Chart.js #5143

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
142e19d
Log errors and skip for buildTask
loicbourgois Jan 13, 2018
58bb9fe
Write gulp error to Chart.js
loicbourgois Jan 13, 2018
2414daa
Remove unused require
loicbourgois Jan 13, 2018
08cf096
Remove error + Proper require fs
loicbourgois Jan 13, 2018
d2327ef
Fix newline
loicbourgois Jan 13, 2018
ecffc82
Refactor
loicbourgois Jan 13, 2018
e490438
Put back browser errors
loicbourgois Jan 13, 2018
e21587b
Use options
loicbourgois Jan 13, 2018
ae0e7e7
Fix intentional error
loicbourgois Jan 13, 2018
c8388d6
Use yargs + Refactor
loicbourgois Jan 13, 2018
251fcc3
remove space
loicbourgois Jan 13, 2018
0b39c03
Fefactor
loicbourgois Jan 13, 2018
6ad9149
Use booleans
loicbourgois Jan 13, 2018
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
5 changes: 5 additions & 0 deletions gulpfile.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,11 @@ function buildTask() {

var errorHandler = function (err) {
util.log(util.colors.red('[Error]'), err.toString());
Copy link
Member

Choose a reason for hiding this comment

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

I would move that line before this.emit('end');, it's only useful when ignoring exceptions

var browserError = 'console.error("Gulp: ' + err.toString() + '")';
fs.writeFileSync(outDir+'Chart.js', browserError);
fs.writeFileSync(outDir+'Chart.min.js', browserError);
fs.writeFileSync(outDir+'Chart.bundle.js', browserError);
fs.writeFileSync(outDir+'Chart.bundle.min.js', browserError);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@simonbrunel if possible, i would like to be able to see the errors directly in the browser. I prefer to work in the editor and the browser only, and not have to check the terminal.

this.emit('end');
}

Expand Down