Skip to content

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