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

Switch to PHPCSStandards/PHP_CodeSniffer #45

Merged
merged 1 commit into from
Dec 8, 2023

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented Dec 2, 2023

⚠️ This is a DRAFT PR on purpose as it references releases which have not yet been tagged. Once the PHPCS 3.8.0 tag is available (which contains the Composer replace directive), this can/should be merged and released ASAP. ⚠️


The Squizlabs repo has been abandoned. The project continues in a fork in the PHPCSStandards organisation.

Includes updating the dev required version of the Composer plugin, which should make sure that this repo already switches over to using the PHPCSStandards version of PHPCS.

Includes updating the suggested version of the Composer plugin to the version which made the same change.

Update: The Composer package name will not change, so while this PR should still be merged at our convenience (after the 3.8.0 release, expected this Friday), we will not need to do a release to unblock end-users.

Ref:

@jrfnl jrfnl added this to the 1.x Next milestone Dec 2, 2023
The Squizlabs repo has been abandoned. The project continues in a fork in the PHPCSStandards organisation.

Ref:
* squizlabs/PHP_CodeSniffer 3932
@jrfnl jrfnl force-pushed the feature/switch-to-fork-of-phpcs branch from cc52c8e to 7d68e2c Compare December 7, 2023 02:51
@jrfnl jrfnl marked this pull request as ready for review December 7, 2023 02:52
@jrfnl jrfnl merged commit 1e22e89 into master Dec 8, 2023
4 of 5 checks passed
@jrfnl jrfnl deleted the feature/switch-to-fork-of-phpcs branch December 8, 2023 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant