jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/361113 )
Change subject: build: Updating mediawiki/mediawiki-codesniffer to 0.9.0 ...................................................................... build: Updating mediawiki/mediawiki-codesniffer to 0.9.0 Change-Id: Ie6489c84c576fe4f0162f5c39e0100dc9c83f90f --- M .gitignore M VectorV2Template.php M composer.json M phpcs.xml 4 files changed, 26 insertions(+), 19 deletions(-) Approvals: Hashar: Looks good to me, approved jenkins-bot: Verified diff --git a/.gitignore b/.gitignore index 53bbca6..087740f 100644 --- a/.gitignore +++ b/.gitignore @@ -18,7 +18,7 @@ sftp-config.json # Building & testing -node_modules/ +/node_modules # Composer /vendor diff --git a/VectorV2Template.php b/VectorV2Template.php index 9b203aa..5aeec5f 100644 --- a/VectorV2Template.php +++ b/VectorV2Template.php @@ -114,7 +114,7 @@ if ( $this->data['title'] != '' ) { ?> <h1 id="firstHeading" class="firstHeading" lang="<?php $this->text( 'pageLanguage' ); ?>"><?php - $this->html( 'title' ) + $this->html( 'title' ) ?></h1> <?php } ?> @@ -322,7 +322,8 @@ </ul> <?php } else { - echo $content; /* Allow raw HTML block to be defined by extensions */ + // Allow raw HTML block to be defined by extensions + echo $content; } $this->renderAfterPortlet( $name ); @@ -520,17 +521,18 @@ <?php echo $this->makeSearchInput( [ 'id' => 'searchInput' ] ); echo Html::hidden( 'title', $this->get( 'searchtitle' ) ); - // We construct two buttons (for 'go' and 'fulltext' search modes), - // but only one will be visible and actionable at a time (they are - // overlaid on top of each other in CSS). - // * Browsers will use the 'fulltext' one by default (as it's the - // first in tree-order), which is desirable when they are unable - // to show search suggestions (either due to being broken or - // having JavaScript turned off). - // * The mediawiki.searchSuggest module, after doing tests for the - // broken browsers, removes the 'fulltext' button and handles - // 'fulltext' search itself; this will reveal the 'go' button and - // cause it to be used. + /* We construct two buttons (for 'go' and 'fulltext' search modes), + * but only one will be visible and actionable at a time (they are + * overlaid on top of each other in CSS). + * * Browsers will use the 'fulltext' one by default (as it's the + * first in tree-order), which is desirable when they are unable + * to show search suggestions (either due to being broken or + * having JavaScript turned off). + * * The mediawiki.searchSuggest module, after doing tests for the + * broken browsers, removes the 'fulltext' button and handles + * 'fulltext' search itself; this will reveal the 'go' button and + * cause it to be used. + */ echo $this->makeSearchButton( 'fulltext', [ 'id' => 'mw-searchButton', 'class' => 'searchButton mw-fallbackSearchButton' ] diff --git a/composer.json b/composer.json index fe15d13..7e889a4 100644 --- a/composer.json +++ b/composer.json @@ -34,14 +34,14 @@ "installer-name": "VectorV2" }, "require-dev": { - "jakub-onderka/php-parallel-lint": "0.9.*", - "mediawiki/mediawiki-codesniffer": "0.7.2", + "jakub-onderka/php-parallel-lint": "0.9.2", + "mediawiki/mediawiki-codesniffer": "0.9.0", "jakub-onderka/php-console-highlighter": "0.3.2" }, "scripts": { "fix": "phpcbf", "test": [ - "parallel-lint . --exclude vendor", + "parallel-lint . --exclude node_modules --exclude vendor", "phpcs -p -s" ] } diff --git a/phpcs.xml b/phpcs.xml index d81a292..a26dbbe 100644 --- a/phpcs.xml +++ b/phpcs.xml @@ -1,8 +1,13 @@ <?xml version="1.0"?> <ruleset> - <rule ref="vendor/mediawiki/mediawiki-codesniffer/MediaWiki"/> + <rule ref="./vendor/mediawiki/mediawiki-codesniffer/MediaWiki"> + <exclude name="MediaWiki.Commenting.FunctionComment.MissingParamComment"/> + <exclude name="MediaWiki.Commenting.FunctionComment.MissingParamTag"/> + <exclude name="MediaWiki.Commenting.FunctionComment.MissingReturn"/> + </rule> <file>.</file> <arg name="extensions" value="php,php5,inc"/> - <arg name="encoding" value="utf8"/> + <arg name="encoding" value="UTF-8"/> <exclude-pattern>vendor</exclude-pattern> + <exclude-pattern>node_modules</exclude-pattern> </ruleset> -- To view, visit https://gerrit.wikimedia.org/r/361113 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie6489c84c576fe4f0162f5c39e0100dc9c83f90f Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/skins/VectorV2 Gerrit-Branch: master Gerrit-Owner: Umherirrender <umherirrender_de...@web.de> Gerrit-Reviewer: Hashar <has...@free.fr> Gerrit-Reviewer: Legoktm <lego...@member.fsf.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits