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

Support [email protected], PHP 8.4 readiness #144

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

codders
Copy link
Contributor

@codders codders commented Dec 6, 2024

This repository is currently using version 34 of the Mediawiki coding style phpcs rules. It should be upgraded to the latest version.

Besides being simply different, the newer versions of the rules introduce, for example,
MediaWiki.Usage.NullableType.ExplicitNullableTypes, which enforces compliance with coming deprecation rules in PHP 8.4 concerning nullable types. If the code does not remove the soon-to-be-deprecated form of nullable type declarations, the library will no longer be usable in Mediawiki for PHP 8.4 deployments.

Replace existing phpcs rules with the standard mediawiki-codesniffer ruleset.

Bug: T379481

@codders codders force-pushed the feat/upgrade-mediawiki-codesniffer-20241206 branch 2 times, most recently from 80c3b1e to 0bd069e Compare December 9, 2024 09:05
@manicki manicki self-requested a review December 9, 2024 09:56
Test with actively supported PHP versions
@codders codders force-pushed the feat/upgrade-mediawiki-codesniffer-20241206 branch from 0bd069e to 126283e Compare December 10, 2024 14:23
@codders codders merged commit 7bc4a6d into master Dec 11, 2024
7 checks passed
@codders codders deleted the feat/upgrade-mediawiki-codesniffer-20241206 branch December 11, 2024 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants