jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/402933 )

Change subject: Fix return void in MediaWikiFarmer::run
......................................................................


Fix return void in MediaWikiFarmer::run

MediaWikiFarmer::_doWiki returns void, no need to use a return statement
here

Change-Id: Ie41fb46109592efa47ba180c21a86265e27ef7ff
---
M MediaWikiFarmer.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Umherirrender: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/MediaWikiFarmer.php b/MediaWikiFarmer.php
index 80084c8..f5186c0 100644
--- a/MediaWikiFarmer.php
+++ b/MediaWikiFarmer.php
@@ -183,7 +183,7 @@
                        $wiki = strtolower( preg_replace( '/[^[:alnum:_\-]]/', 
'', $wiki ) );
 
                        // now we have a valid wiki name
-                       return $this->_doWiki( $wiki );
+                       $this->_doWiki( $wiki );
 
                } else {
                        throw new MWException(

-- 
To view, visit https://gerrit.wikimedia.org/r/402933
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie41fb46109592efa47ba180c21a86265e27ef7ff
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Farmer
Gerrit-Branch: master
Gerrit-Owner: Umherirrender <umherirrender_de...@web.de>
Gerrit-Reviewer: Umherirrender <umherirrender_de...@web.de>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to