@Andy-MS looks like the other definitions are more updated, but also not sure if they will work as a replacement for the ones we are removing. Any way to test that? Maybe run the current test suite against the other definitions?

After 5 days, no one has reviewed the PR 😞. A maintainer will be reviewing the PR in the next few days and will either merge it or request revisions. Thank you for your patience!

@Andy-MS One or more reviewers has requested changes. Please address their comments. I'll be back once they sign off or you've pushed new commits. Thank you!

@FanaHOVA It looks like chartjs may be for an earlier version of the library? (The version number is missing in the header.) The newer one doesn't seem to have a .Line, .Bar., .Radar, etc. methods on Chart. The latest docs use type: 'line' instead.

@Andy-MS please merge and run the depreciation script.

Ran deprecation script.

@Andy-MS yup, so I'm assuming old versions are unsupported now that this is merged?

@FanaHOVA It should still be possible to install the old chartjs -- you will see a deprecation message but installation should still succeed.
If you want to maintain types for an old version of the package on DefinitelyTyped it would also be possible to create a chart.js/v0 or chart.js/v1 directory containing those typings, similar to node/v0.