Page MenuHomePhabricator

Release mediawiki-codesniffer v37.0.0
Closed, ResolvedPublic

Description

T278653: Release mediawiki-codesniffer v36.0.0 | NoneYet

Given we generally do about one of these every three months, it's coming up to time. We don't have an official regular release process (yet, see T266890).

* build: Update php-parallel-lint/php-parallel-lint (Umherirrender)
* Allow configuring prefixes in PrefixedGlobalFunctionsSniff (DannyS712)
* Minor clean up to LowerCamelFunctionsNameSniff (DannyS712)
* AssertCountSniff: allow manual `count` on both sides (DannyS712)
* RedundantVarNameSniff: also check static properties (DannyS712)
* PrefixedGlobalFunctionsSniff: drop `ef` from default prefixes (DannyS712)
* UnsortedUseStatementsSniff: add early returns (DannyS712)
* FunctionCommentSniff: also skip {@inheritDoc} (DannyS712)
* DocumentationTypeTrait: complain about `type` as a type (DannyS712)
* Add Gerrit report format (Gergő Tisza)
* Add fix reporting to Gerrit robot comment reporter (Gergő Tisza)

Event Timeline

The autofix for T220719: Standardize return type hint spacing will result in autofix changes to a whole bunch of files (every phpunit test file already includes void typehints enforced by a different sniff, so those will all be changed, so its likely that all extensions will have at least some files autofixed and thus the patches will require manual review and +2 instead of libup merging automatically). On the one hand, it may make sense to wait until that patch is merged to include it in this release, but on the other hand it may make sense to have that separately because it'll require manual reviewing. Not sure

There are also some pending patches that I would hope can get reviewed before the next release, including those for

Change 704969 had a related patch set uploaded (by Jforrester; author: Jforrester):

[mediawiki/tools/codesniffer@master] Release v37.0.0

https://gerrit.wikimedia.org/r/704969

Change 704969 merged by jenkins-bot:

[mediawiki/tools/codesniffer@master] Release v37.0.0

https://gerrit.wikimedia.org/r/704969

Change 705734 had a related patch set uploaded (by Jforrester; author: Jforrester):

[labs/libraryupgrader/config@master] Bump mediawiki-codesniffer from 36.0.0 to 37.0.0

https://gerrit.wikimedia.org/r/705734

Jdforrester-WMF claimed this task.

Demonstration patch to prove it works: https://gerrit.wikimedia.org/r/c/mediawiki/services/function-schemata/+/705731

LibUp will get this out everywhere.

Change 705734 merged by jenkins-bot:

[labs/libraryupgrader/config@master] Bump mediawiki-codesniffer from 36.0.0 to 37.0.0

https://gerrit.wikimedia.org/r/705734