jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/375840 )
Change subject: build: Updating mediawiki/mediawiki-codesniffer to 0.12.0
......................................................................
build: Updating mediawiki/mediawiki-codesniffer to 0.12.0
The following sniffs are failing and were disabled:
* MediaWiki.Commenting.FunctionComment.MissingParamComment
Also added "composer fix" command.
Change-Id: Ie065bfb1129835599e8ef4fdb24159157f2033b9
---
M composer.json
M phpcs.xml
M src/ScopedCallback.php
M tests/ScopedCallbackTest.php
4 files changed, 13 insertions(+), 6 deletions(-)
Approvals:
Krinkle: Looks good to me, approved
jenkins-bot: Verified
diff --git a/composer.json b/composer.json
index 7ccd7a5..8b8284a 100644
--- a/composer.json
+++ b/composer.json
@@ -10,7 +10,9 @@
}
],
"autoload": {
- "classmap": ["src/"]
+ "classmap": [
+ "src/"
+ ]
},
"require": {
"php": ">=5.5.9"
@@ -18,13 +20,16 @@
"require-dev": {
"jakub-onderka/php-parallel-lint": "0.9.2",
"phpunit/phpunit": "4.8.27",
- "mediawiki/mediawiki-codesniffer": "0.7.2"
+ "mediawiki/mediawiki-codesniffer": "0.12.0"
},
"scripts": {
"test": [
"parallel-lint . --exclude vendor",
"phpunit",
"phpcs -p -s"
+ ],
+ "fix": [
+ "phpcbf"
]
}
}
diff --git a/phpcs.xml b/phpcs.xml
index 06c43bc..6100ee9 100644
--- a/phpcs.xml
+++ b/phpcs.xml
@@ -1,6 +1,8 @@
<?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" />
+ </rule>
<file>.</file>
<arg name="encoding" value="UTF-8"/>
<arg name="extensions" value="php"/>
diff --git a/src/ScopedCallback.php b/src/ScopedCallback.php
index 16d475e..ab9e1be 100644
--- a/src/ScopedCallback.php
+++ b/src/ScopedCallback.php
@@ -50,7 +50,7 @@
* Trigger a scoped callback and destroy it.
* This is the same as just setting it to null.
*
- * @param ScopedCallback $sc
+ * @param ScopedCallback &$sc
*/
public static function consume( ScopedCallback &$sc = null ) {
$sc = null;
@@ -59,7 +59,7 @@
/**
* Destroy a scoped callback without triggering it.
*
- * @param ScopedCallback $sc
+ * @param ScopedCallback &$sc
*/
public static function cancel( ScopedCallback &$sc = null ) {
if ( $sc ) {
diff --git a/tests/ScopedCallbackTest.php b/tests/ScopedCallbackTest.php
index 7daa93f..f4c1f72 100644
--- a/tests/ScopedCallbackTest.php
+++ b/tests/ScopedCallbackTest.php
@@ -43,7 +43,7 @@
public function testParams() {
$params = [ 'foo', 'bar', 'baz' ];
- $sc = new ScopedCallback( function( /*...*/ ) {
+ $sc = new ScopedCallback( function () {
$this->assertSame( [ 'foo', 'bar', 'baz' ],
func_get_args() );
}, $params );
ScopedCallback::consume( $sc );
--
To view, visit https://gerrit.wikimedia.org/r/375840
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie065bfb1129835599e8ef4fdb24159157f2033b9
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/libs/ScopedCallback
Gerrit-Branch: master
Gerrit-Owner: Libraryupgrader <[email protected]>
Gerrit-Reviewer: Krinkle <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
