Page MenuHomePhabricator

Coding conventions
Closed, InvalidPublic

Description

Use phpcs/phpcbf to lint PHP code.
https://github.com/wikimedia/mediawiki-tools-codesniffer

  • composer.json
  • phpcs.xml
  • Run phpcbf (composer fix).
  • Fix problems cannot be autofixed.

Event Timeline

Change 285143 had a related patch set uploaded (by 小璋丸):
refs #T133633 Follow coding conventions.

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

Change 285151 had a related patch set uploaded (by 小璋丸):
refs T133633, Fix second key, which confict with global one.

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

Change 285152 had a related patch set uploaded (by 小璋丸):
refs T133633, Follow coding conventions (Line: 1~210)

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

Change 285153 had a related patch set uploaded (by 小璋丸):
refs T133633, Follow coding conventions (Line: 201~345)

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

@raymondwu: Do you know where the bug tracker for https://www.mediawiki.org/wiki/Extension:QuickGV is located? There is no information on the wiki page...
If you are its maintainer and want a project in Phabricator, please see https://www.mediawiki.org/wiki/Phabricator/Creating_and_renaming_projects#New_projects - thanks!

I'm the maintainer. But I don't have privilege to create project.

Change 287056 had a related patch set uploaded (by 小璋丸):
refs #T133633 Add space characters and syntax highlight.

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

raymondwu renamed this task from QuickGV coding conventions fix to Coding conventions, syntax highlighting, and version string..May 26 2016, 6:47 AM
raymondwu updated the task description. (Show Details)

Change 285143 had a related patch set uploaded (by Aklapper):
Indent with spaces and update version string.

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

Change 285143 merged by Raimond Spekking:
Indent with spaces and update version string.

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

raymondwu renamed this task from Coding conventions, syntax highlighting, and version string. to Coding conventions.Aug 12 2016, 3:59 AM
raymondwu updated the task description. (Show Details)

Change 334405 had a related patch set uploaded (by 小璋丸):
Follow coding conventions.

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

Removing task assignee due to inactivity, as this open task has been assigned to the same person for more than two years (see the emails sent to the task assignee on Oct27 and Nov23). Please assign this task to yourself again if you still realistically [plan to] work on this task - it would be welcome.
(See https://www.mediawiki.org/wiki/Bug_management/Assignee_cleanup for tips how to best manage your individual work in Phabricator.)