chartjs / chartjs-plugin-zoom

Zoom and pan plugin for Chart.js
MIT License
579 stars 322 forks source link

Reduce the number of dependencies #812

Open bavoco opened 3 months ago

bavoco commented 3 months ago

Doing an npm install after cloning this project downloads 1658 dependencies, there are also many warnings about deprecated and vulnerable packages. Maybe it's a good idea to update, remove, or switch some dependencies to actively maintained projects.

Is this a direction worth pursuing?

NotTsunami commented 3 months ago

Holy moly, I had never looked at the package.json of this plugin before and was just using it blindly. This is an insane tree of dependencies for a plugin to another library. I can't imagine this initiative would get blocked either. I'd be willing to contribute time towards this initiative on my spare time. Can happily help with linting related dependencies!

EDIT: I can probably take on updating testing related dependencies too. Will try to get a PR up for both by this weekend.

bavoco commented 3 months ago

I didn't go further on this because there was no response to the first two yet. Feel free to take them up!