Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Composer: allow for the 1.0.0 version of the Composer PHPCS plugin #738

Merged

Conversation

jrfnl
Copy link
Collaborator

@jrfnl jrfnl commented Jan 5, 2023

The Composer PHPCS plugin has released its 1.0.0 version. 🎉

Important:
I've widened the version constraints for the plugin, instead of bumping it.

The reason for this is to prevent conflicts with end-user projects/other external PHPCS standards which may also require(-dev) the plugin, but may not (yet) have updated their constraints for the plugin. If the version would have been bumped instead of widened, those users would get an unsolvable conflict during the composer install run (unless they require-dev the plugin for the root project, but then, that's exactly what we don't want them to do as external standards managing the versions of the plugins should be more reliable).

The version constraints now set cover all released versions which support external standards properly.

Ref: https://github.com/PHPCSStandards/composer-installer/releases/tag/v1.0.0

The Composer PHPCS plugin has released its 1.0.0 version. 🎉

Important:
I've _widened_ the version constraints for the plugin, instead of _bumping_ it.

The reason for this is to prevent conflicts with end-user projects/other external PHPCS standards which may also require(-dev) the plugin, but may not (yet) have updated _their_ constraints for the plugin.
If the version would have been bumped instead of widened, those users would get an unsolvable conflict during the `composer install` run (unless they `require-dev` the plugin for the root project, but then, that's exactly what we _don't_ want them to do as external standards managing the versions of the plugins should  be more reliable).

The version constraints now set cover all released versions which support external standards properly.

Ref: https://github.com/PHPCSStandards/composer-installer/releases/tag/v1.0.0
@jrfnl jrfnl added this to the 2.3.4 milestone Jan 5, 2023
@jrfnl jrfnl requested a review from a team as a code owner January 5, 2023 12:52
Copy link
Contributor

@GaryJones GaryJones left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@GaryJones GaryJones merged commit 22733aa into develop Jan 5, 2023
@GaryJones GaryJones deleted the feature/update-for-1.0-release-of-composer-phpcs-plugin branch January 5, 2023 13:03
@kasparsd
Copy link

Thanks for the fix!

Is there a plan to release a stable version with these changes? It appears that the current diff is relatively small https://github.com/Automattic/VIP-Coding-Standards/compare/2.3.3..22733aa560f0f8855b848475770e4f800157e7f4 and contains mostly dev related changes.

@GaryJones
Copy link
Contributor

Is there a plan to release a stable version with these changes?

We're looking into that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants