[go: up one dir, main page]
Page MenuHomePhabricator
Create Task
Maniphest T177810

Fix up PHPCS violations that are excluded in .phpcs.xml
Open, Needs TriagePublic
Actions

Description

PHPCodeSniffer helps us stick to same coding standards across MediaWiki and its extensions.
In MediaWiki core and various MW extensions, we have PHP CS rules in a file called .phpcs.xml that are excluded because the code repository has various violations. In most cases, these can't be fixed automatically.

Remove a sniff from the .phpcs.xml file, run composer phpcs or phpcs -p -s, get a list of violations, provide patches in Wikimedia Gerrit to fix the violations. If you fix all violations for one sniff, also commit the removal of the exclusion from .phpcs.xml too.

Fix some (define a number here?) of these up.

https://tools.wmflabs.org/phpcs/

Details

SubjectRepoBranchLines +/-
Remove ParamNameNoMatch phpcs sniff and fix errorsmediawiki/extensions/Cargomaster+53 -26
Namespace classes and give each its own filemediawiki/extensions/PageTriagemaster+1 K -1 K
Customize query in gerrit

Related Objects
Search...

Event Timeline

Reedy created this task.Oct 10 2017, 1:07 AM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 10 2017, 1:07 AM
Aklapper added projects: MediaWiki-General, MediaWiki-extensions-General.Oct 10 2017, 11:44 AM
MtDu subscribed.Oct 17 2017, 9:12 PM

Parent task maybe? https://phabricator.wikimedia.org/T170580 @Reedy

Reedy added a comment.EditedOct 18 2017, 4:46 AM

That could be a subtask of this, though this was designed as a GCI task specifically

Aklapper added subtasks: T170580: Re-enable codesniffer sniffs disabled in TextExtracts, T170583: Re-enable codesniffer sniffs disabled in PageImages, T170579: Re-enable codesniffer sniffs disabled in MobileFrontend.Nov 20 2017, 11:02 PM
Aklapper added a project: Epic.Nov 20 2017, 11:08 PM
Aklapper updated the task description. (Show Details)
Florian subscribed.Nov 29 2017, 10:08 PM

I think 5-10 changes should be reasonable, wouldn't it? @Reedy Do you want to mentor this task together with me? :D

Reedy added a comment.Nov 29 2017, 10:28 PM

You mean, 5-10 separate changes?

Aklapper added a comment.Nov 30 2017, 12:09 PM

Also see existing subtasks (especially for improving the task description). To get a list of excluded sniffs per MediaWiki extension repository:
find /home/user/git/mw/extensions/ -type f -name "*phpcs.xml" -printf "%h\n" -execdir grep -H '<exclude name=' '{}' +

Jdlrobson closed subtask T170580: Re-enable codesniffer sniffs disabled in TextExtracts as Resolved.Nov 30 2017, 10:28 PM
Florian added a comment.Dec 11 2017, 9:21 AM
In T177810#3798447, @Reedy wrote:

You mean, 5-10 separate changes?

No, I mean 5-10 modified things inside one change to complete :) Sorry for the confusion.

Aklapper added a comment.Dec 11 2017, 11:35 AM

Given the amount of work we've asked for in the existing 2 or 3 subtasks already I'd put that higher than 5-10.

Pppery added a subtask: T177809: Fix multiple PHP class declarations in one file.Dec 19 2017, 1:18 AM
jayvdb moved this task from Proposed tasks to Imported in GCI Site on the Google-Code-in-2017 board.Dec 19 2017, 10:12 AM
Pppery added a parent task: T162936: Improve developer happiness in reading web maintained extensions.Dec 21 2017, 3:38 PM
Jdlrobson closed subtask T170579: Re-enable codesniffer sniffs disabled in MobileFrontend as Resolved.Jan 12 2018, 4:40 PM
Kizule closed subtask T183676: Re-enable codesniffer sniffs disabled in TweetANew as Resolved.Jan 28 2018, 7:56 PM
Pppery closed subtask T170583: Re-enable codesniffer sniffs disabled in PageImages as Resolved.Mar 5 2018, 5:58 PM
Samwilson renamed this task from Fix up PHPCS violations that are excluded in phpcs.xml to Fix up PHPCS violations that are excluded in .phpcs.xml.Jun 6 2018, 11:28 PM
gerritbot subscribed.Jun 7 2018, 2:42 AM

Change 437660 had a related patch set uploaded (by Samwilson; owner: Samwilson):
[mediawiki/extensions/PageTriage@master] Namespace classes and give each its own file

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

gerritbot added a project: Patch-For-Review.Jun 7 2018, 2:42 AM
gerritbot added a comment.Jun 12 2018, 12:39 AM

Change 437660 merged by jenkins-bot:
[mediawiki/extensions/PageTriage@master] Namespace classes and give each its own file

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

Jdlrobson removed a parent task: T162936: Improve developer happiness in reading web maintained extensions.Jul 9 2018, 4:27 PM
Legoktm updated the task description. (Show Details)Aug 19 2018, 7:12 AM
Aklapper removed a project: Patch-For-Review.Oct 4 2018, 11:09 AM
gerritbot added a comment.Oct 12 2018, 9:11 AM

Change 466839 had a related patch set uploaded (by Samwilson; owner: Samwilson):
[mediawiki/extensions/Cargo@master] Remove ParamNameNoMatch phpcs sniff and fix errors

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

gerritbot added a project: Patch-For-Review.Oct 12 2018, 9:11 AM
gerritbot added a comment.Dec 20 2018, 3:35 AM

Change 466839 merged by jenkins-bot:
[mediawiki/extensions/Cargo@master] Remove ParamNameNoMatch phpcs sniff and fix errors

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

Maintenance_bot removed a project: Patch-For-Review.May 22 2019, 3:00 PM
DannyS712 subscribed.Apr 11 2020, 1:30 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits