Hacker News Reader

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.

They 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.

I'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/integrat...

Reply on news.ycombinator.com