4 ms·
Why it still works, no need to refactor the code for the sake of refactoring if this is free work. Change to ES modules would be welcome though.
by ergo14 7y ago
Why it still works, no need to refactor the code for the sake of refactoring if this is free work.
Change to ES modules would be welcome though.
- vimslayer 7y agoThey seem to be using Rollup for builds, which supports ESM output, so they wouldn't even need to do any refactoring for that. Couple of configuration changes would suffice.
- benmccann 7y agoI'd be happy to review any PR to improve our rollup & package.json config as long as it's backwards compatible with the documented methods of using Chart.js: https://www.chartjs.org/docs/latest/getting-started/integration.html https://www.chartjs.org/docs/latest/getting-started/integrat...