[BUG] version 2.7.0 is breaking SEMVER specs.

[jorge-autogravity]

According to http://semver.org/ any breaking change is cause of bumping the semver major version. In this case chart.js should be 3.0.0 instead of 2.7.0

That versioning inconsistency broke a feature in my web app last build.

[simonbrunel]

@jorge-autogravity can you be more explicit about the exact breaking change?

[jorge-autogravity]

@simonbrunel the breaking change mentioned on https://github.com/chartjs/Chart.js/releases/tag/v2.7.0

[simonbrunel]

Agree, #4079 should have been postponed until 3.0.0 - really sorry for that. I guess we didn't release it as 3.0.0 because we have lot of pending work that we want to finish until Chart.js 3, so we released 2.7.0 since many people were asking for.

@etimberg should we delay any PR that introduces breaking changes to the next major release? that might become complicated to maintain because PRs will diverge quickly from master. Or maybe should we consider to target a new major version as soon as we want to introduce a breaking change? meaning that we will not support a major version for very long.

@jorge-autogravity you marked this ticket as BUG, what do you suggest?

[jorge-autogravity]

@simonbrunel I think its too late to revert.. unless you want to publish 2.8.0 reverting the breaking change.
Anyways we locked the version in our end. I don't want to add more work to you. Just wanted to point out an area of improvement.

Thanks for building and maintaining this OSS project. It has been very helpful to me and my team.

[etimberg]

That breaking change is hard because it essentially reverts back to the v2.0.0 behaviour that changed at a previous minor version.

In general we probably need a v3 release soon so that we can remove a lot of code that maintains backwards compatibility.

[jeff-kilbride]

Also, aspectRatio isn't listed as an option in the docs, even though that's the suggested method for reverting to pre 2.7.0 behavior (unless I'm missing it).

I found an example in the original pull request: #4079 (review)

[etimberg]

Closing since this is quite old and not relevant to the current version (2.9.x). The next released version will be v3.0