Change Details

During code review we discovered https://gerrit.wikimedia.org/r/#/c/319067/7/tests/phpunit/MobileFormatterTest.php,unified

During code review we discovered a linting inconsistency https://gerrit.wikimedia.org/r/#/c/319067/7/tests/phpunit/MobileFormatterTest.php,unified ``` private function makeSectionHeading( $heading, $innerHtml, $sectionNumber=1 ) { ``` VS ``` private function makeSectionHeading( $heading, $innerHtml, $sectionNumber = 1 ) { ``` We should enforce and standardise on one of these styles

During code review we discovered a linting inconsistency https://gerrit.wikimedia.org/r/#/c/319067/7/tests/phpunit/MobileFormatterTest.php,unified ``` private function makeSectionHeading( $heading, $innerHtml, $sectionNumber=1 ) { ``` VS ``` private function makeSectionHeading( $heading, $innerHtml, $sectionNumber = 1 ) { ``` We should enforce and standardise on one of these styles