Closed timnolte closed 5 months ago
Thank you for taking the time to report this.
Looks like we dropped the ball on this one :disappointed:
I've updated the release notes for the v1 release to clearly state the backward-breaking change.
I'm not 100% convinced change needs to be made to the README... @jrfnl do you have opinions on the matter?
eh, you obviously had a pretty big release on your hands so these things can happen. Thanks for updating the release notes! 👍🏼
The README was updated in the same PR as the namespace rename.
I personally don't think that the README needs to document outdated usage, so IMO, the README as-is, is fine. Especially considering every previous release was a pre-1.0 release and 1.0 should be the "truth" going forward.
I do very much agree that this change should have been highlighted more clearly in the changelog/release notes. Thank you @timnolte for calling that out and @Potherca for making the update!
@Potherca Should the branch rename from master
to main
maybe also be listed with the breaking changes ?
Problem/Motivation
Nothing in the README or the Release notes clearly indicated a BREAKING CHANGE to the namespace rename. Upgrading the package and trying to look through documentation to find what might have change proved almost impossible. The README needs to clearly indicate with it's own major heading and highlighting that this change was made. To be fair, I recognize that this release by nature of it being a major SemVer release could be taken that way though in this day and age it's not always the case. ;-)
Expected behaviour
Reading through the README or the Release notes should have very clear BREAKING CHANGE notices to help people fix the issue of failing runs due this significant change.
Actual behaviour
And since the package name didn't change it is not obvious right away without a lot digging what happened.
Steps to reproduce
composer.json
script that runsDealerdirect\\Composer\\Plugin\\Installers\\PHPCodeSniffer\\Plugin::run
after package install/update.Proposed changes
In the README section for "Calling the plugin directly" this namespace change should be clearly called out.
Environment
Output of
vendor/bin/phpcs --config-show
:Tested against
main
branch?main
branch.