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: raise the minimum supported PHPCS version to 3.9.0 and remove work-arounds #561

Merged
merged 4 commits into from
Feb 26, 2024

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented Feb 26, 2024

Composer: raise the minimum supported PHPCS version to 3.9.0

... to benefit from the syntax support for PHP 8.3 which has been added to PHPCS itself.

Includes updating references to the PHPCS version whenever relevant throughout the codebase.

BCFile::getDeclarationName(): defer to upstream

No BC-layer is needed at this time with a minimum supported PHPCS version of PHPCS 3.9.0.

BCFile::getClassProperties(): defer to upstream

No BC-layer is needed at this time with a minimum supported PHPCS version of PHPCS 3.9.0.

IsShortArrayOrList::isShortArrayBracket(): remove BC layer

... which is no longer needed now support for PHPCS < 3.9.0 has been dropped.

... to benefit from the syntax support for PHP 8.3 which has been added to PHPCS itself.

Includes updating references to the PHPCS version whenever relevant throughout the codebase.
No BC-layer is needed at this time with a minimum supported PHPCS version of PHPCS 3.9.0.
No BC-layer is needed at this time with a minimum supported PHPCS version of PHPCS 3.9.0.
... which is no longer needed now support for PHPCS < 3.9.0 has been dropped.
@jrfnl jrfnl added this to the 1.0.x Next milestone Feb 26, 2024
@jrfnl jrfnl merged commit 1513597 into develop Feb 26, 2024
56 checks passed
@jrfnl jrfnl deleted the feature/composer-update-minimum-phpcs-version branch February 26, 2024 09:55
@jrfnl jrfnl mentioned this pull request Mar 17, 2024
14 tasks
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.

1 participant