We now have phpcs passing. We should improve that by removing all exceptions and severity 0 where possible
Description
Details
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Open | DannyS712 | T171115 Remove phpcs exceptions and severity 0 from mediawiki-config | |||
Resolved | Reedy | T162835 Enforce our PHP coding standards on operations/mediawiki-config via composer? |
Event Timeline
Change 773863 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable and fix SingleSpaceBeforeSingleLineComment
Change 773864 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable and fix UnusedGlobalVariables
Change 773865 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: narrow some exclusions only needed for cirrusTest.php
Change 773966 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: clean up MWConfigCacheGenerator and enable rules
Change 773863 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable and fix SingleSpaceBeforeSingleLineComment
Mentioned in SAL (#wikimedia-operations) [2022-03-28T13:13:55Z] <lucaswerkmeister-wmde@deploy1002> Synchronized wmf-config/InitialiseSettings-labs.php: Config: [[gerrit:773863|phpcs: enable and fix SingleSpaceBeforeSingleLineComment (T171115)]] (phpcs.xml will be synced with next patch) (duration: 01m 01s)
Change 773864 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable passing rule UnusedGlobalVariables
Mentioned in SAL (#wikimedia-operations) [2022-03-28T13:17:55Z] <lucaswerkmeister-wmde@deploy1002> Synchronized phpcs.xml: Config: [[gerrit:773864|phpcs: enable passing rule UnusedGlobalVariables (T171115)]] (includes phpcs.xml change from previous sync) (duration: 00m 56s)
Change 773865 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: narrow some exclusions only needed for cirrusTest.php
Mentioned in SAL (#wikimedia-operations) [2022-03-28T13:21:58Z] <lucaswerkmeister-wmde@deploy1002> Synchronized tests/cirrusTest.php: Config: [[gerrit:773865|phpcs: narrow some exclusions only needed for cirrusTest.php (T171115)]] (1/2) (duration: 00m 56s)
Mentioned in SAL (#wikimedia-operations) [2022-03-28T13:23:03Z] <lucaswerkmeister-wmde@deploy1002> Synchronized phpcs.xml: Config: [[gerrit:773865|phpcs: narrow some exclusions only needed for cirrusTest.php (T171115)]] (2/2) (duration: 00m 55s)
Change 775005 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable rules that are already passing
Change 775426 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: rename test files to match class names
Change 775427 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable and fix PropertyDocumentation.MissingVar
Change 773966 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: clean up MWConfigCacheGenerator and enable rules
Change 775005 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable rules that are already passing
Change 775426 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: rename test files to match class names
Change 775427 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable and fix PropertyDocumentation.MissingVar
Mentioned in SAL (#wikimedia-operations) [2022-03-31T20:46:14Z] <thcipriani@deploy1002> Synchronized phpcs.xml: Config (noop): [[gerrit:775427|phpcs: enable and fix PropertyDocumentation.MissingVar (T171115)]] [[gerrit:775426|phpcs: rename test files to match class names (T171115)]] [[gerrit:775005|phpcs: enable rules that are already passing (T171115)]] (duration: 00m 49s)
Mentioned in SAL (#wikimedia-operations) [2022-03-31T20:47:41Z] <thcipriani@deploy1002> Synchronized src/StaticSiteConfiguration.php: Config (noop -- comment change): [[gerrit:775427|phpcs: enable and fix PropertyDocumentation.MissingVar (T171115)]] (duration: 00m 50s)
Change 775937 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable PropertyDocumentation.MissingDocumentationProtected
Change 775938 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable FunctionComment.MissingParamTag
Change 775939 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: add @covers tags to test files
Change 796352 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: move DisallowYodaConditions exclusion inline
Change 796353 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable FunctionComment.MissingDocumentationPrivate
Change 796354 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable PropertyDocumentation.WrongStyle
Change 796355 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable and suppress DuplicateClassName.Found
Change 796356 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable PropertyDocumentation.MissingDocumentationPrivate
Change 775937 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable PropertyDocumentation.MissingDocumentationProtected
Change 775938 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable FunctionComment.MissingParamTag
Change 775939 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: add @covers tags to test files
Change 796352 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: move DisallowYodaConditions exclusion inline
Change 796353 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable FunctionComment.MissingDocumentationPrivate
Change 796354 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable PropertyDocumentation.WrongStyle
Change 796357 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable and suppress ClassMatchesFilename.NotMatch
Change 796355 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable and suppress DuplicateClassName.Found
Change 796356 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable PropertyDocumentation.MissingDocumentationPrivate
Change 796357 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable and suppress ClassMatchesFilename.NotMatch
Change 796358 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: move ForbiddenFunctions.extract exclusion inline
Change 796359 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: move ForbiddenFunctions.exec exclusion inline
Change 796358 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: move ForbiddenFunctions.extract exclusion inline
Change 796360 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable passing AssignmentInControlStructures rule
Change 796359 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: move ForbiddenFunctions.exec exclusion inline
Change 802840 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: move Misleading$wgDebugLogFile exclusion inline
Change 802841 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable and fix FunctionComment.WrongStyle
Change 802842 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable and configure ValidGlobalName.allowedPrefix
Change 802946 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable and configure PrefixedGlobalFunctions.allowedPrefix
Change 802947 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: move Misleading$wgConf exclusion inline
Change 796360 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: move AssignmentInControlStructures exclusion inline
Change 802840 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: move Misleading$wgDebugLogFile exclusion inline
Change 802841 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable and fix FunctionComment.WrongStyle
Change 802842 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable and configure ValidGlobalName.allowedPrefix
Change 802946 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable and configure PrefixedGlobalFunctions.allowedPrefix
Change 802947 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable and fix MisleadingGlobalNames.Misleading$wgConf
Mentioned in SAL (#wikimedia-operations) [2022-06-14T20:31:10Z] <urbanecm@deploy1002> Synchronized wmf-config/: phpcs cleanups (T171115; no-op for production) (duration: 03m 38s)
Mentioned in SAL (#wikimedia-operations) [2022-06-14T20:34:39Z] <urbanecm@deploy1002> Synchronized multiversion/: phpcs cleanups (T171115; no-op for production) (duration: 03m 28s)
Mentioned in SAL (#wikimedia-operations) [2022-06-14T20:37:55Z] <urbanecm@deploy1002> Synchronized w/: phpcs cleanups (T171115; no-op for production) (duration: 03m 15s)
Mentioned in SAL (#wikimedia-operations) [2022-06-14T20:41:36Z] <urbanecm@deploy1002> Synchronized docroot/: phpcs cleanups (T171115; no-op for production) (duration: 03m 41s)
Change 805430 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: start to fix SpaceBeforeSingleLineComment.NewLineComment
Change 805430 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: start to fix SpaceBeforeSingleLineComment.NewLineComment
Change 805431 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: fix more SpaceBeforeSingleLineComment.NewLineComment
Mentioned in SAL (#wikimedia-operations) [2022-06-14T21:51:27Z] <urbanecm@deploy1002> Synchronized docroot/: ca3b94f2d9bc755d92839e5e69072615ea9008df: phpcs: start to fix SpaceBeforeSingleLineComment.NewLineComment (T171115) (duration: 03m 38s)
Mentioned in SAL (#wikimedia-operations) [2022-06-14T21:54:58Z] <urbanecm@deploy1002> Synchronized multiversion/: ca3b94f2d9bc755d92839e5e69072615ea9008df: phpcs: start to fix SpaceBeforeSingleLineComment.NewLineComment (T171115) (duration: 03m 29s)
Mentioned in SAL (#wikimedia-operations) [2022-06-14T21:58:27Z] <urbanecm@deploy1002> Synchronized rpc/: ca3b94f2d9bc755d92839e5e69072615ea9008df: phpcs: start to fix SpaceBeforeSingleLineComment.NewLineComment (T171115) (duration: 03m 31s)
Mentioned in SAL (#wikimedia-operations) [2022-06-14T22:02:00Z] <urbanecm@deploy1002> Synchronized src/: ca3b94f2d9bc755d92839e5e69072615ea9008df: phpcs: start to fix SpaceBeforeSingleLineComment.NewLineComment (T171115) (duration: 03m 32s)
Mentioned in SAL (#wikimedia-operations) [2022-06-14T22:05:19Z] <urbanecm@deploy1002> Synchronized w/: ca3b94f2d9bc755d92839e5e69072615ea9008df: phpcs: start to fix SpaceBeforeSingleLineComment.NewLineComment (T171115) (duration: 03m 18s)
Change 805431 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: fix more SpaceBeforeSingleLineComment.NewLineComment
Mentioned in SAL (#wikimedia-operations) [2022-06-14T22:15:03Z] <urbanecm@deploy1002> Synchronized wmf-config/: e3fe6c04c95717f0f914bbfa366f5f827f392b6b: phpcs: fix more SpaceBeforeSingleLineComment.NewLineComment (T171115) (duration: 03m 39s)
Change 805432 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: move SpaceBeforeSingleLineComment.NewLineComment exclusions
Change 806248 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] phpcs: enable PrefixedGlobalFunctions.allowedPrefix and rename functions
Change 805432 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: move SpaceBeforeSingleLineComment.NewLineComment exclusions
Change 806249 had a related patch set uploaded (by DannyS712; author: DannyS712):
[operations/mediawiki-config@master] MWRealm.php: remove unused getRealmSpecificFilename()
Mentioned in SAL (#wikimedia-operations) [2022-06-16T20:23:29Z] <cjming@deploy1002> Synchronized wmf-config/: Config: [[gerrit:805432|phpcs: move SpaceBeforeSingleLineComment.NewLineComment exclusions (T171115)]] (duration: 03m 22s)
Mentioned in SAL (#wikimedia-operations) [2022-06-16T20:27:22Z] <cjming@deploy1002> Synchronized phpcs.xml: Config: [[gerrit:805432|phpcs: move SpaceBeforeSingleLineComment.NewLineComment exclusions (T171115)]] (duration: 03m 27s)
Change 806248 merged by jenkins-bot:
[operations/mediawiki-config@master] phpcs: enable PrefixedGlobalFunctions.allowedPrefix and rename functions
Mentioned in SAL (#wikimedia-operations) [2022-06-16T20:42:01Z] <thcipriani@deploy1002> Started scap: Config: [[gerrit:806248|phpcs: enable PrefixedGlobalFunctions.allowedPrefix and rename functions (T171115)]]
Change 806249 merged by jenkins-bot:
[operations/mediawiki-config@master] MWRealm.php: remove unused getRealmSpecificFilename()
Mentioned in SAL (#wikimedia-operations) [2022-06-16T20:58:58Z] <thcipriani@deploy1002> Finished scap: Config: [[gerrit:806248|phpcs: enable PrefixedGlobalFunctions.allowedPrefix and rename functions (T171115)]] (duration: 16m 57s)
Mentioned in SAL (#wikimedia-operations) [2022-06-16T21:06:28Z] <thcipriani@deploy1002> Synchronized multiversion/MWRealm.php: Config: [[gerrit:806249|MWRealm.php: remove unused getRealmSpecificFilename() (T171115)]] (duration: 03m 35s)