Umherirrender has uploaded a new change for review. ( https://gerrit.wikimedia.org/r/371003 )
Change subject: Improve some parameter docs ...................................................................... Improve some parameter docs Change-Id: Ic78a889eb7374731a282312c9eadb701483ae298 --- M composer.json M includes/UserPreferencesChangeHandler.php M phpcs.xml 3 files changed, 6 insertions(+), 8 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Popups refs/changes/03/371003/1 diff --git a/composer.json b/composer.json index 55507be..91c6304 100644 --- a/composer.json +++ b/composer.json @@ -6,7 +6,7 @@ }, "scripts": { "test": [ - "parallel-lint . --exclude vendor", + "parallel-lint . --exclude vendor --exclude node_modules", "phpcs -p -s" ], "fix": [ diff --git a/includes/UserPreferencesChangeHandler.php b/includes/UserPreferencesChangeHandler.php index ccd3770..ee80b48 100644 --- a/includes/UserPreferencesChangeHandler.php +++ b/includes/UserPreferencesChangeHandler.php @@ -74,7 +74,7 @@ * @param array $formData Form data submitted by user * @param PreferencesForm $form A preferences form * @param User $user Logged-in user - * @param boolean &$result Variable defining is form save successful + * @param bool &$result Variable defining is form save successful * @param array $oldUserOptions Old user options array */ public static function onPreferencesFormPreSave( diff --git a/phpcs.xml b/phpcs.xml index 7255308..1a3c9e9 100644 --- a/phpcs.xml +++ b/phpcs.xml @@ -1,17 +1,15 @@ <?xml version="1.0"?> <ruleset> - <rule ref="./vendor/mediawiki/mediawiki-codesniffer/MediaWiki"> - </rule> + <rule ref="./vendor/mediawiki/mediawiki-codesniffer/MediaWiki"/> <file>.</file> <arg name="extensions" value="php,php5,inc" /> - <arg name="encoding" value="utf8" /> - <exclude-pattern>vendor</exclude-pattern> + <arg name="encoding" value="UTF-8" /> <rule ref="Squiz.Functions.FunctionDeclarationArgumentSpacing"> <properties> <property name="equalsSpacing" value="1" /> - <property name="requiredSpacesAfterOpen" value="1" /> - <property name="requiredSpacesBeforeClose" value="1" /> + <property name="requiredSpacesAfterOpen" value="1" /> + <property name="requiredSpacesBeforeClose" value="1" /> </properties> </rule> </ruleset> -- To view, visit https://gerrit.wikimedia.org/r/371003 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic78a889eb7374731a282312c9eadb701483ae298 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/Popups Gerrit-Branch: master Gerrit-Owner: Umherirrender <umherirrender_de...@web.de> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits