chartjs / Chart.js / 4981

97%

push

travis-ci

Migrate from Browserify to rollup (#5904)

Browserify isn't optimal bundling Chart.js because it adds too many internal wrappers, doesn't handle external/global dependencies and doesn't provide a way to generate ESM builds. Therefore, it seems the right choice to switch to rollup, so move all the build process in `rollup.config.js` and make Gulp to execute `rollup -c`.

We also had to switch to Terser instead of UglifyJS because this last one contains a breaking bug. Note that tests now use the exact same rollup config as our builds (the minified one) to ensure that the bundling and minification steps don't break anything. Finally, replace the `gulp watch` task by `gulp build --watch` to be consistent with the other `unittest` and `docs` watching syntax.

2637 of 3030 branches covered (87.03%)

8 of 8 new or added lines in 4 files covered. (100.0%)

5012 of 5219 relevant lines covered (96.03%)

2629.51 hits per line