date:20171104

[MediaWiki-commits] [Gerrit] mediawiki...GettingStarted[master]: Update git.wikimedia.org URL to Phabricator

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Update git.wikimedia.org URL to Phabricator
..


Update git.wikimedia.org URL to Phabricator

Bug: T139089
Change-Id: I88bda19857f1e85c82657c888b928c17bda21219
---
M tests/browser/README.md
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/tests/browser/README.md b/tests/browser/README.md
index 98534f8..5fca59e 100644
--- a/tests/browser/README.md
+++ b/tests/browser/README.md
@@ -2,4 +2,4 @@
 
 Run the tests with `bundle exec cucumber`.
 
-See 
 for 
instructions on how to configure your development environment for running the 
tests.
+See  
for instructions on how to configure your development environment for running 
the tests.

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I88bda19857f1e85c82657c888b928c17bda21219
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/GettingStarted
Gerrit-Branch: master
Gerrit-Owner: TerraCodes 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: Mattflaschen 
Gerrit-Reviewer: Phuedx 
Gerrit-Reviewer: Swalling 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...EditAccount[master]: Update git.wikimedia.org URLs to Phabricator

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Update git.wikimedia.org URLs to Phabricator
..


Update git.wikimedia.org URLs to Phabricator

Bug: T139089
Change-Id: I7e44972d70e309934aa758a9276312d621d212fe
---
M SpecialEditAccount_body.php
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/SpecialEditAccount_body.php b/SpecialEditAccount_body.php
index 761f24e..b3315e9 100644
--- a/SpecialEditAccount_body.php
+++ b/SpecialEditAccount_body.php
@@ -737,7 +737,7 @@
 
/**
 * Copypasta from pre-1.23 /includes/GlobalFunctions.php
-* @see 
https://git.wikimedia.org/commitdiff/mediawiki%2Fcore/118567a4ba0ded669f43a58713733cab915afe39
+* @see 
https://phabricator.wikimedia.org/rMW118567a4ba0ded669f43a58713733cab915afe39
 */
public static function generateToken( $salt = '' ) {
$salt = serialize( $salt );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7e44972d70e309934aa758a9276312d621d212fe
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/EditAccount
Gerrit-Branch: master
Gerrit-Owner: TerraCodes 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...GettingStarted[master]: git.wikimedia.org -> phab

2017-11-04 Thread TerraCodes (Code Review)

TerraCodes has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389301 )

Change subject: git.wikimedia.org -> phab
..

git.wikimedia.org -> phab

Bug: T139089
Change-Id: I88bda19857f1e85c82657c888b928c17bda21219
---
M tests/browser/README.md
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/GettingStarted 
refs/changes/01/389301/1

diff --git a/tests/browser/README.md b/tests/browser/README.md
index 98534f8..5fca59e 100644
--- a/tests/browser/README.md
+++ b/tests/browser/README.md
@@ -2,4 +2,4 @@
 
 Run the tests with `bundle exec cucumber`.
 
-See 
 for 
instructions on how to configure your development environment for running the 
tests.
+See  
for instructions on how to configure your development environment for running 
the tests.

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I88bda19857f1e85c82657c888b928c17bda21219
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/GettingStarted
Gerrit-Branch: master
Gerrit-Owner: TerraCodes 

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

[MediaWiki-commits] [Gerrit] mediawiki...AbuseFilter[master]: AbuseFilter block range should not exceed $wgBlockCIDRLimit

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: AbuseFilter block range should not exceed $wgBlockCIDRLimit
..


AbuseFilter block range should not exceed $wgBlockCIDRLimit

This patch introduces a config variable for the range block sizes.
It changes the default IPv6 block size from /16 to /19 using the
same reasoning as  Ia25e156fd8234519c4d74f1d41d93f94a313ce14

Using a config var (as opposed to hardcoded range size) allows
future changes proposed in T179454 to make the range size vary
for different IPs, based on the actual subnet they belong to.

Bug: T179455
Bug: T179456
Change-Id: I8dfa17f553a7af524f0a11c0fd51c48773e27be5
---
M extension.json
M includes/AbuseFilter.class.php
2 files changed, 15 insertions(+), 2 deletions(-)

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



diff --git a/extension.json b/extension.json
index 4bbb34f..8a9dfc7 100644
--- a/extension.json
+++ b/extension.json
@@ -250,7 +250,11 @@
"AbuseFilterLogIPMaxAge": 7776000,
"AbuseFilterProfile": false,
"AbuseFilterRuntimeProfile": false,
-   "AbuseFilterSlowFilterRuntimeLimit": 500
+   "AbuseFilterSlowFilterRuntimeLimit": 500,
+   "AbuseFilterRangeBlockSize" : {
+   "IPv4": 16,
+   "IPv6": 19
+   }
},
"load_composer_autoloader": true,
"manifest_version": 1
diff --git a/includes/AbuseFilter.class.php b/includes/AbuseFilter.class.php
index 0487452..1e936b0 100644
--- a/includes/AbuseFilter.class.php
+++ b/includes/AbuseFilter.class.php
@@ -1447,12 +1447,21 @@
];
break;
case 'rangeblock':
+   global $wgAbuseFilterRangeBlockSize, 
$wgBlockCIDRLimit;
+
+   $ip = $wgRequest->getIP();
+   if ( IP::isIPv6( $ip ) ) {
+   $CIDRsize = max( 
$wgAbuseFilterRangeBlockSize['IPv6'], $wgBlockCIDRLimit['IPv6'] );
+   } else {
+   $CIDRsize = max( 
$wgAbuseFilterRangeBlockSize['IPv4'], $wgBlockCIDRLimit['IPv4'] );
+   }
+   $blockCIDR = $ip . '/' . $CIDRsize;
self::doAbuseFilterBlock(
[
'desc' => $rule_desc,
'number' => $rule_number
],
-   IP::sanitizeRange( $wgRequest->getIP() 
. '/16' ),
+   IP::sanitizeRange( $blockCIDR ),
'1 week',
false
);

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8dfa17f553a7af524f0a11c0fd51c48773e27be5
Gerrit-PatchSet: 4
Gerrit-Project: mediawiki/extensions/AbuseFilter
Gerrit-Branch: master
Gerrit-Owner: Huji 
Gerrit-Reviewer: Jackmcbarn 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: Matěj Suchánek 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Language: Update git.wikimedia.org URLs to point to Phabricator

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Language: Update git.wikimedia.org URLs to point to Phabricator
..


Language: Update git.wikimedia.org URLs to point to Phabricator

Bug: T139089
Change-Id: I38492e0c8e90f3418bcb831f086fbe6ea2b319b5
---
M languages/Language.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  TerraCodes: Looks good to me, but someone else must approve
  Legoktm: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/languages/Language.php b/languages/Language.php
index 8517f26..81564ab 100644
--- a/languages/Language.php
+++ b/languages/Language.php
@@ -167,7 +167,7 @@
 * The form is '/(?:([strong ltr codepoint])|([strong rtl 
codepoint]))/u' .
 *
 * Generated by UnicodeJS (see tools/strongDir) from the UCD; see
-* https://git.wikimedia.org/summary/unicodejs.git .
+* https://phabricator.wikimedia.org/diffusion/GUJS/ .
 */
// @codingStandardsIgnoreStart
// @codeCoverageIgnoreStart

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I38492e0c8e90f3418bcb831f086fbe6ea2b319b5
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: TerraCodes 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: TerraCodes 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...EditAccount[master]: git.wikimedia.org -> phab

2017-11-04 Thread TerraCodes (Code Review)

TerraCodes has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389300 )

Change subject: git.wikimedia.org -> phab
..

git.wikimedia.org -> phab

Bug: T139089
Change-Id: I7e44972d70e309934aa758a9276312d621d212fe
---
A .SpecialEditAccount_body.php.un~
M SpecialEditAccount_body.php
A SpecialEditAccount_body.php~
3 files changed, 747 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/EditAccount 
refs/changes/00/389300/1

diff --git a/.SpecialEditAccount_body.php.un~ b/.SpecialEditAccount_body.php.un~
new file mode 100644
index 000..61ace91
--- /dev/null
+++ b/.SpecialEditAccount_body.php.un~
Binary files differ
diff --git a/SpecialEditAccount_body.php b/SpecialEditAccount_body.php
index 761f24e..b3315e9 100644
--- a/SpecialEditAccount_body.php
+++ b/SpecialEditAccount_body.php
@@ -737,7 +737,7 @@
 
/**
 * Copypasta from pre-1.23 /includes/GlobalFunctions.php
-* @see 
https://git.wikimedia.org/commitdiff/mediawiki%2Fcore/118567a4ba0ded669f43a58713733cab915afe39
+* @see 
https://phabricator.wikimedia.org/rMW118567a4ba0ded669f43a58713733cab915afe39
 */
public static function generateToken( $salt = '' ) {
$salt = serialize( $salt );
diff --git a/SpecialEditAccount_body.php~ b/SpecialEditAccount_body.php~
new file mode 100644
index 000..b3315e9
--- /dev/null
+++ b/SpecialEditAccount_body.php~
@@ -0,0 +1,746 @@
+
+ * @date 2008-09-17
+ * @copyright Copyright © 2008 Łukasz Garczewski, Wikia Inc.
+ * @license http://www.gnu.org/copyleft/gpl.html GNU General Public License 
2.0 or later
+ */
+
+class EditAccount extends SpecialPage {
+   public $mUser = null;
+   public $mStatus = null;
+   public $mStatusMsg;
+   public $mStatusMsg2 = null;
+   public $mTempUser = null;
+
+   /**
+* Constructor -- set up the new special page
+*/
+   public function __construct() {
+   parent::__construct( 'EditAccount', 'editaccount' );
+   }
+
+   public function doesWrites() {
+   return true;
+   }
+
+   /**
+* Group this special page under the correct header in 
Special:SpecialPages.
+*
+* @return string
+*/
+   function getGroupName() {
+   return 'users';
+   }
+
+   /**
+* Special page description shown on Special:SpecialPages -- different 
for
+* privileged users and mortals
+*
+* @return string Special page description
+*/
+   function getDescription() {
+   if ( $this->getUser()->isAllowed( 'editaccount' ) ) {
+   return $this->msg( 'editaccount' )->plain();
+   } else {
+   return $this->msg( 'editaccount-general-description' 
)->plain();
+   }
+   }
+
+   /**
+* Show the special page
+*
+* @param string|null $par Parameter (user name) passed to the page or 
null
+*/
+   public function execute( $par ) {
+   $out = $this->getOutput();
+   $request = $this->getRequest();
+   $user = $this->getUser();
+
+   // Redirect mortals to Special:CloseAccount
+   if ( !$user->isAllowed( 'editaccount' ) ) {
+   //throw new PermissionsError( 'editaccount' );
+   $out->redirect( SpecialPage::getTitleFor( 
'CloseAccount' )->getFullURL() );
+   }
+
+   // Show a message if the database is in read-only mode
+   if ( wfReadOnly() ) {
+   $out->readOnlyPage();
+   return;
+   }
+
+   // If user is blocked, s/he doesn't need to access this page
+   if ( $user->isBlocked() ) {
+   throw new UserBlockedError( $user->mBlock );
+   }
+
+   // Set page title and other stuff
+   $this->setHeaders();
+
+   // Special:EditAccount is a fairly stupid page title
+   $out->setPageTitle( $this->getDescription() );
+
+   // Get name to work on. Subpage is supported, but form submit 
name trumps
+   $userName = $request->getVal( 'wpUserName', $par );
+   $action = $request->getVal( 'wpAction' );
+
+   if( $userName !== null ) {
+   // Got a name, clean it up
+   $userName = str_replace( '_', ' ', trim( $userName ) );
+   // User names begin with a capital letter
+   $userName = $this->getLanguage()->ucfirst( $userName );
+
+   // Check if user name is an existing user
+   if ( User::isValidUserName( $userName ) ) {
+   $this->mUser = User::newFromName( $userName );
+

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: git.wikimedia.org -> phab

2017-11-04 Thread TerraCodes (Code Review)

TerraCodes has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389299 )

Change subject: git.wikimedia.org -> phab
..

git.wikimedia.org -> phab

Bug: T139089
Change-Id: I38492e0c8e90f3418bcb831f086fbe6ea2b319b5
---
M languages/Language.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/99/389299/1

diff --git a/languages/Language.php b/languages/Language.php
index 8517f26..81564ab 100644
--- a/languages/Language.php
+++ b/languages/Language.php
@@ -167,7 +167,7 @@
 * The form is '/(?:([strong ltr codepoint])|([strong rtl 
codepoint]))/u' .
 *
 * Generated by UnicodeJS (see tools/strongDir) from the UCD; see
-* https://git.wikimedia.org/summary/unicodejs.git .
+* https://phabricator.wikimedia.org/diffusion/GUJS/ .
 */
// @codingStandardsIgnoreStart
// @codeCoverageIgnoreStart

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I38492e0c8e90f3418bcb831f086fbe6ea2b319b5
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: TerraCodes 

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

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Ensure WCAG level AA contrast

2017-11-04 Thread VolkerE (Code Review)

VolkerE has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389298 )

Change subject: Ensure WCAG level AA contrast
..

Ensure WCAG level AA contrast

Increasing contrast on span.stats slightly to meet
WCAG level AA contrast minimum requirement on possible
background color `#f9f9f9`.

Change-Id: If033cd61bd81dbd5773810b1a944f20a9bf2e58d
---
M resources/src/mediawiki/mediawiki.debug.less
1 file changed, 9 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/98/389298/1

diff --git a/resources/src/mediawiki/mediawiki.debug.less 
b/resources/src/mediawiki/mediawiki.debug.less
index dec0571..00faf84 100644
--- a/resources/src/mediawiki/mediawiki.debug.less
+++ b/resources/src/mediawiki/mediawiki.debug.less
@@ -16,6 +16,14 @@
width: 100%;
table-layout: fixed;
 
+   tr {
+   background-color: #fff;
+
+   &:nth-child( even ) {
+   background-color: #f9f9f9;
+   }
+   }
+
td,
th {
padding: 4px 10px;
@@ -30,15 +38,7 @@
}
 
span.stats {
-   color: #808080;
-   }
-   }
-
-   tr {
-   background-color: #fff;
-
-   &:nth-child( even ) {
-   background-color: #f9f9f9;
+   color: #727272;
}
}
}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If033cd61bd81dbd5773810b1a944f20a9bf2e58d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: VolkerE 

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

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Align icon variants with WikimediaUI color palette

2017-11-04 Thread VolkerE (Code Review)

VolkerE has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389297 )

Change subject: Align icon variants with WikimediaUI color palette
..

Align icon variants with WikimediaUI color palette

Follow-up to I067973dca580eddc09b770362183d7a5b1634b7c

Bug: T146799
Change-Id: Idd02de3883c8c5a971dff657d002f4c9e09c88eb
---
M skin.json
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/skins/MinervaNeue 
refs/changes/97/389297/1

diff --git a/skin.json b/skin.json
index 174a670..23d999a 100644
--- a/skin.json
+++ b/skin.json
@@ -149,11 +149,11 @@
"class": "ResourceLoaderImageModule",
"variants": {
"gray": {
-   "color": "#BBB",
+   "color": "#54595d",
"global": true
},
"invert": {
-   "color": "#FFF",
+   "color": "#fff",
"global": true
}
},

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idd02de3883c8c5a971dff657d002f4c9e09c88eb
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/skins/MinervaNeue
Gerrit-Branch: master
Gerrit-Owner: VolkerE 

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

[MediaWiki-commits] [Gerrit] mediawiki...Vector[master]: Align VectorMenu items and unify `padding` with tabs

2017-11-04 Thread VolkerE (Code Review)

VolkerE has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389296 )

Change subject: Align VectorMenu items and unify `padding` with tabs
..

Align VectorMenu items and unify `padding` with tabs

Aligning VectorMenu items horizontally with menu handle and
set `padding` consistently to `8px` – taking into account
different `font-size`.
Also removing unnecessary duplication of `margin-top` as `margin`
is set in same rule.

Bug: T179782
Change-Id: I30e9b4e50589ef09a0e5dcb0e6c4486e7779a39d
---
M components/tabs.less
M components/watchstar.less
2 files changed, 6 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/skins/Vector 
refs/changes/96/389296/1

diff --git a/components/tabs.less b/components/tabs.less
index ab5f96b..d49f8c3 100644
--- a/components/tabs.less
+++ b/components/tabs.less
@@ -90,8 +90,8 @@
/* For IE6, overridden later to display:block by modern 
browsers */
display: inline-block;
height: 1.9em;
-   padding-left: 0.5em;
-   padding-right: 0.5em;
+   padding-left: 0.625em; // equals `8px` at computed 
`font-size` of `12.8px`
+   padding-right: 0.625em;
color: @menu-link-color;
cursor: pointer;
font-size: 0.8em;
@@ -155,7 +155,7 @@
height: 2.5em;
// `padding-right` >= `1px` effectively moves the "background border" 
outside of the element to act like a real
// border. It is necessary for `div.vectorMenu div.menu` dropdown to 
align well.
-   padding-right: 0.5em; // equals `8px` as visually harmonically with 
`padding-left` in `div.vectorMenu h3 span`
+   padding-right: 0.5em; // equals `8px` at computed `font-size` of `14px` 
as visually harmonically with `padding-left` in `div.vectorMenu h3 span`
margin-right: -1px;
 
.vector-nav-directionality & {
@@ -167,7 +167,7 @@
span {
display: block;
font-size: 0.8em;
-   padding-left: 0.7em;
+   padding-left: 0.625em;
padding-top: 1.25em;
padding-right: 16px;
font-weight: normal;
@@ -242,7 +242,7 @@
 /* OVERRIDDEN BY COMPLIANT BROWSERS */
 div.vectorMenu li a {
display: inline-block;
-   padding: 0.5em;
+   padding: 0.625em;
white-space: nowrap;
color: @menu-link-color;
cursor: pointer;
diff --git a/components/watchstar.less b/components/watchstar.less
index a4315ec..da51133 100644
--- a/components/watchstar.less
+++ b/components/watchstar.less
@@ -9,10 +9,9 @@
margin: 0;
padding: 0;
display: block;
-   width: 26px;
+   width: 28px;
/* This hides the text but shows the background image */
padding-top: 3.1em;
-   margin-top: 0;
/* Only applied in IE6 */
_margin-top: -0.8em;
height: 0;

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I30e9b4e50589ef09a0e5dcb0e6c4486e7779a39d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/skins/Vector
Gerrit-Branch: master
Gerrit-Owner: VolkerE 

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

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Update the installphp.sh script to use a supported PHP version

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Update the installphp.sh script to use a supported PHP version
..


Update the installphp.sh script to use a supported PHP version

It has been stuck on 5.4 since it was added to the repository
in 9e25a8e32e6d. It also contained some typos and updating
the version required multiple changes, so that has been fixed.

Bug: T165079
Change-Id: I30b4d6d0701dc60651d3c07fd364312a78a45800
---
M maintenance/dev/installphp.sh
1 file changed, 15 insertions(+), 14 deletions(-)

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



diff --git a/maintenance/dev/installphp.sh b/maintenance/dev/installphp.sh
index d26ffa6..1e3d410 100755
--- a/maintenance/dev/installphp.sh
+++ b/maintenance/dev/installphp.sh
@@ -12,13 +12,14 @@
exit 0
 fi
 
-TAR=php5.4-latest.tar.gz
-PHPURL="http://snaps.php.net/$TAR";
+VER=5.6.32
+TAR="php-$VER.tar.gz"
+PHPURL="https://secure.php.net/get/$TAR/from/this/mirror";
 
 cd "$DEV"
 
-echo "Preparing to download and install a local copy of PHP 5.4, note that 
this can take some time to do."
-echo "If you wish to avoid re-doing this for uture dev installations of 
MediaWiki we suggest installing php in ~/.mediawiki/php"
+echo "Preparing to download and install a local copy of PHP $VER, note that 
this can take some time to do."
+echo "If you wish to avoid re-doing this for future dev installations of 
MediaWiki we suggest installing php in ~/.mediawiki/php"
 echo -n "Install PHP in ~/.mediawiki/php [y/N]: "
 read INSTALLINHOME
 
@@ -31,27 +32,27 @@
;;
 esac
 
-# Some debain-like systems bundle wget but not curl, some other systems
+# Some debian-like systems bundle wget but not curl, some other systems
 # like os x bundle curl but not wget... use whatever is available
-echo -n "Downloading PHP 5.4"
+echo -n "Downloading PHP $VER"
 if command -v wget &>/dev/null; then
-   echo "- using wget"
-   wget "$PHPURL"
+   echo " - using wget"
+   wget -O "$TAR" "$PHPURL"
 elif command -v curl &>/dev/null; then
-   echo "- using curl"
-   curl -O "$PHPURL"
+   echo " - using curl"
+   curl "$PHPURL" -L -o "$TAR"
 else
-   echo "- aborting"
+   echo " - aborting"
echo "Could not find curl or wget." >&2;
exit 1;
 fi
 
-echo "Extracting php 5.4"
+echo "Extracting php $VER"
 tar -xzf "$TAR"
 
-cd php5.4-*/
+cd "php-$VER/"
 
-echo "Configuring and installing php 5.4 in $PREFIX"
+echo "Configuring and installing php $VER in $PREFIX"
 ./configure --prefix="$PREFIX"
 make
 make install

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I30b4d6d0701dc60651d3c07fd364312a78a45800
Gerrit-PatchSet: 4
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Mainframe98 
Gerrit-Reviewer: Daniel Friesen 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: Mainframe98 
Gerrit-Reviewer: Parent5446 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] operations/puppet[production]: mediawiki_vagrant: Update defined(Class['role::labs_vagrant'...

2017-11-04 Thread Paladox (Code Review)

Paladox has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389295 )

Change subject: mediawiki_vagrant: Update defined(Class['role::labs_vagrant'] 
to role::deprecated::labsvagrant 
..

mediawiki_vagrant: Update defined(Class['role::labs_vagrant'] to 
role::deprecated::labsvagrant 

Change-Id: I89d61682a53cdef6689152af212ee71834539125
---
0 files changed, 0 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/95/389295/1


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I89d61682a53cdef6689152af212ee71834539125
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Paladox 

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

[MediaWiki-commits] [Gerrit] mediawiki...BlueSpicePagesVisited[master]: Add php-parallel-lint, grunt-jsonlint and grunt-banana-checker

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389294 )

Change subject: Add php-parallel-lint, grunt-jsonlint and grunt-banana-checker
..

Add php-parallel-lint, grunt-jsonlint and grunt-banana-checker

jsonlint and banana-checker will test for valid i18n files
php-lint will test for valid php files

Change-Id: I8e6ed5330e752b5afeea9eec6a47903a7feecd94
---
A .gitignore
A Gruntfile.js
M composer.json
A package.json
4 files changed, 43 insertions(+), 1 deletion(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BlueSpicePagesVisited 
refs/changes/94/389294/1

diff --git a/.gitignore b/.gitignore
new file mode 100644
index 000..db8a779
--- /dev/null
+++ b/.gitignore
@@ -0,0 +1,3 @@
+/node_modules
+/vendor
+/composer.lock
diff --git a/Gruntfile.js b/Gruntfile.js
new file mode 100644
index 000..ea67e84
--- /dev/null
+++ b/Gruntfile.js
@@ -0,0 +1,19 @@
+module.exports = function ( grunt ) {
+   grunt.loadNpmTasks( 'grunt-jsonlint' );
+   grunt.loadNpmTasks( 'grunt-banana-checker' );
+
+   var conf = grunt.file.readJSON( 'extension.json' );
+   grunt.initConfig( {
+   banana: conf.MessagesDirs,
+   jsonlint: {
+   all: [
+   '**/*.json',
+   '!node_modules/**',
+   '!vendor/**'
+   ]
+   }
+   } );
+
+   grunt.registerTask( 'test', [ 'jsonlint', 'banana' ] );
+   grunt.registerTask( 'default', 'test' );
+};
diff --git a/composer.json b/composer.json
index db434bb..09453f4 100644
--- a/composer.json
+++ b/composer.json
@@ -9,5 +9,14 @@
"autoload": {
"psr-4": {
}
+   },
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules"
+   ]
}
-}
\ No newline at end of file
+}
diff --git a/package.json b/package.json
new file mode 100644
index 000..10fa9bc
--- /dev/null
+++ b/package.json
@@ -0,0 +1,11 @@
+{
+   "private": true,
+   "scripts": {
+   "test": "grunt test"
+   },
+   "devDependencies": {
+   "grunt": "1.0.1",
+   "grunt-banana-checker": "0.6.0",
+   "grunt-jsonlint": "1.1.0"
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8e6ed5330e752b5afeea9eec6a47903a7feecd94
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpicePagesVisited
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...BlueSpiceInterWikiLinks[master]: Add php-parallel-lint, grunt-jsonlint and grunt-banana-checker

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/389293 )

Change subject: Add php-parallel-lint, grunt-jsonlint and grunt-banana-checker
..


Add php-parallel-lint, grunt-jsonlint and grunt-banana-checker

jsonlint and banana-checker will test for valid i18n files
php-lint will test for valid php files

Change-Id: I8cbd37bd80fcc4f29195d13080735506ce20bae8
---
A .gitignore
A Gruntfile.js
M composer.json
A package.json
4 files changed, 43 insertions(+), 1 deletion(-)

Approvals:
  Umherirrender: Verified; Looks good to me, approved



diff --git a/.gitignore b/.gitignore
new file mode 100644
index 000..db8a779
--- /dev/null
+++ b/.gitignore
@@ -0,0 +1,3 @@
+/node_modules
+/vendor
+/composer.lock
diff --git a/Gruntfile.js b/Gruntfile.js
new file mode 100644
index 000..ea67e84
--- /dev/null
+++ b/Gruntfile.js
@@ -0,0 +1,19 @@
+module.exports = function ( grunt ) {
+   grunt.loadNpmTasks( 'grunt-jsonlint' );
+   grunt.loadNpmTasks( 'grunt-banana-checker' );
+
+   var conf = grunt.file.readJSON( 'extension.json' );
+   grunt.initConfig( {
+   banana: conf.MessagesDirs,
+   jsonlint: {
+   all: [
+   '**/*.json',
+   '!node_modules/**',
+   '!vendor/**'
+   ]
+   }
+   } );
+
+   grunt.registerTask( 'test', [ 'jsonlint', 'banana' ] );
+   grunt.registerTask( 'default', 'test' );
+};
diff --git a/composer.json b/composer.json
index a2c6378..f0bb5f6 100644
--- a/composer.json
+++ b/composer.json
@@ -9,5 +9,14 @@
"autoload": {
"psr-4": {
}
+   },
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules"
+   ]
}
-}
\ No newline at end of file
+}
diff --git a/package.json b/package.json
new file mode 100644
index 000..10fa9bc
--- /dev/null
+++ b/package.json
@@ -0,0 +1,11 @@
+{
+   "private": true,
+   "scripts": {
+   "test": "grunt test"
+   },
+   "devDependencies": {
+   "grunt": "1.0.1",
+   "grunt-banana-checker": "0.6.0",
+   "grunt-jsonlint": "1.1.0"
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8cbd37bd80fcc4f29195d13080735506ce20bae8
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceInterWikiLinks
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...BlueSpicePagesVisited[master]: Add php-parallel-lint, grunt-jsonlint and grunt-banana-checker

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/389294 )

Change subject: Add php-parallel-lint, grunt-jsonlint and grunt-banana-checker
..


Add php-parallel-lint, grunt-jsonlint and grunt-banana-checker

jsonlint and banana-checker will test for valid i18n files
php-lint will test for valid php files

Change-Id: I8e6ed5330e752b5afeea9eec6a47903a7feecd94
---
A .gitignore
A Gruntfile.js
M composer.json
A package.json
4 files changed, 43 insertions(+), 1 deletion(-)

Approvals:
  Umherirrender: Verified; Looks good to me, approved



diff --git a/.gitignore b/.gitignore
new file mode 100644
index 000..db8a779
--- /dev/null
+++ b/.gitignore
@@ -0,0 +1,3 @@
+/node_modules
+/vendor
+/composer.lock
diff --git a/Gruntfile.js b/Gruntfile.js
new file mode 100644
index 000..ea67e84
--- /dev/null
+++ b/Gruntfile.js
@@ -0,0 +1,19 @@
+module.exports = function ( grunt ) {
+   grunt.loadNpmTasks( 'grunt-jsonlint' );
+   grunt.loadNpmTasks( 'grunt-banana-checker' );
+
+   var conf = grunt.file.readJSON( 'extension.json' );
+   grunt.initConfig( {
+   banana: conf.MessagesDirs,
+   jsonlint: {
+   all: [
+   '**/*.json',
+   '!node_modules/**',
+   '!vendor/**'
+   ]
+   }
+   } );
+
+   grunt.registerTask( 'test', [ 'jsonlint', 'banana' ] );
+   grunt.registerTask( 'default', 'test' );
+};
diff --git a/composer.json b/composer.json
index db434bb..09453f4 100644
--- a/composer.json
+++ b/composer.json
@@ -9,5 +9,14 @@
"autoload": {
"psr-4": {
}
+   },
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules"
+   ]
}
-}
\ No newline at end of file
+}
diff --git a/package.json b/package.json
new file mode 100644
index 000..10fa9bc
--- /dev/null
+++ b/package.json
@@ -0,0 +1,11 @@
+{
+   "private": true,
+   "scripts": {
+   "test": "grunt test"
+   },
+   "devDependencies": {
+   "grunt": "1.0.1",
+   "grunt-banana-checker": "0.6.0",
+   "grunt-jsonlint": "1.1.0"
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8e6ed5330e752b5afeea9eec6a47903a7feecd94
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpicePagesVisited
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...BlueSpiceInterWikiLinks[master]: Add php-parallel-lint, grunt-jsonlint and grunt-banana-checker

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389293 )

Change subject: Add php-parallel-lint, grunt-jsonlint and grunt-banana-checker
..

Add php-parallel-lint, grunt-jsonlint and grunt-banana-checker

jsonlint and banana-checker will test for valid i18n files
php-lint will test for valid php files

Change-Id: I8cbd37bd80fcc4f29195d13080735506ce20bae8
---
A .gitignore
A Gruntfile.js
M composer.json
A package.json
4 files changed, 43 insertions(+), 1 deletion(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BlueSpiceInterWikiLinks 
refs/changes/93/389293/1

diff --git a/.gitignore b/.gitignore
new file mode 100644
index 000..db8a779
--- /dev/null
+++ b/.gitignore
@@ -0,0 +1,3 @@
+/node_modules
+/vendor
+/composer.lock
diff --git a/Gruntfile.js b/Gruntfile.js
new file mode 100644
index 000..ea67e84
--- /dev/null
+++ b/Gruntfile.js
@@ -0,0 +1,19 @@
+module.exports = function ( grunt ) {
+   grunt.loadNpmTasks( 'grunt-jsonlint' );
+   grunt.loadNpmTasks( 'grunt-banana-checker' );
+
+   var conf = grunt.file.readJSON( 'extension.json' );
+   grunt.initConfig( {
+   banana: conf.MessagesDirs,
+   jsonlint: {
+   all: [
+   '**/*.json',
+   '!node_modules/**',
+   '!vendor/**'
+   ]
+   }
+   } );
+
+   grunt.registerTask( 'test', [ 'jsonlint', 'banana' ] );
+   grunt.registerTask( 'default', 'test' );
+};
diff --git a/composer.json b/composer.json
index a2c6378..f0bb5f6 100644
--- a/composer.json
+++ b/composer.json
@@ -9,5 +9,14 @@
"autoload": {
"psr-4": {
}
+   },
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules"
+   ]
}
-}
\ No newline at end of file
+}
diff --git a/package.json b/package.json
new file mode 100644
index 000..10fa9bc
--- /dev/null
+++ b/package.json
@@ -0,0 +1,11 @@
+{
+   "private": true,
+   "scripts": {
+   "test": "grunt test"
+   },
+   "devDependencies": {
+   "grunt": "1.0.1",
+   "grunt-banana-checker": "0.6.0",
+   "grunt-jsonlint": "1.1.0"
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8cbd37bd80fcc4f29195d13080735506ce20bae8
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceInterWikiLinks
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] pywikibot/core[master]: noreferences.py: Localise name of template reflist on serbia...

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: noreferences.py: Localise name of template reflist on serbian 
language
..


noreferences.py: Localise name of template reflist on serbian language

Bug: T176019
Change-Id: Idddcc117495684f4cd7efd9c8f24b7b6f9e67501
---
M scripts/noreferences.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Framawiki: Looks good to me, but someone else must approve
  Merlijn van Deen: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/scripts/noreferences.py b/scripts/noreferences.py
index b0b5b7d..7f0a213 100755
--- a/scripts/noreferences.py
+++ b/scripts/noreferences.py
@@ -418,7 +418,7 @@
 'ru': [u'Reflist', u'Ref-list', u'Refs', u'Sources',
u'Примечания', u'Список примечаний',
u'Сноска', u'Сноски'],
-'sr': ['Reflist'],
+'sr': ['Reflist', 'Референце', 'Извори'],
 'szl': [u'Przipisy', u'Připisy'],
 'th': [u'รายการอ้างอิง'],
 'zh': [u'Reflist', u'RefFoot', u'NoteFoot'],

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Idddcc117495684f4cd7efd9c8f24b7b6f9e67501
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Zoranzoki21 
Gerrit-Reviewer: Framawiki 
Gerrit-Reviewer: John Vandenberg 
Gerrit-Reviewer: Merlijn van Deen 
Gerrit-Reviewer: Zoranzoki21 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] integration/config[master]: Archive the ThemeDesigner extension

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389292 )

Change subject: Archive the ThemeDesigner extension
..

Archive the ThemeDesigner extension

Bug: T178754
Change-Id: If4fcc0ea427f9119d1618f9fa2bf0bb072b39e6a
---
M zuul/layout.yaml
1 file changed, 1 insertion(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/integration/config 
refs/changes/92/389292/2

diff --git a/zuul/layout.yaml b/zuul/layout.yaml
index d569a0e..a15aed7 100644
--- a/zuul/layout.yaml
+++ b/zuul/layout.yaml
@@ -5737,8 +5737,7 @@
 
   - name: mediawiki/extensions/ThemeDesigner
 template:
- - name: extension-unittests-generic
- - name: mwgate-npm
+ - name: archived
 
   - name: mediawiki/extensions/TheWikipediaLibrary
 template:

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If4fcc0ea427f9119d1618f9fa2bf0bb072b39e6a
Gerrit-PatchSet: 2
Gerrit-Project: integration/config
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] integration/config[master]: Whitelist Gerrit Patch Uploader to CI Whitelist.

2017-11-04 Thread Zoranzoki21 (Code Review)

Zoranzoki21 has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389291 )

Change subject: Whitelist Gerrit Patch Uploader to CI Whitelist.
..

Whitelist Gerrit Patch Uploader to CI Whitelist.

Change-Id: I91116f694aba89bfca4569f2c500028c49b785cb
---
M zuul/layout.yaml
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/integration/config 
refs/changes/91/389291/2

diff --git a/zuul/layout.yaml b/zuul/layout.yaml
index d569a0e..b4976b8 100644
--- a/zuul/layout.yaml
+++ b/zuul/layout.yaml
@@ -104,6 +104,7 @@
 | florian\.schmidt\.stargatewissen@gmail\.com
 | fomafix@googlemail\.com
 | framawiki@tools\.wmflabs\.org
+| gerritpatchuploader@gmail\.com
 | gianiskido@gmail\.com
 | git@samsmith\.io
 | geofbot@gmail\.com
@@ -392,6 +393,7 @@
- ^florian\.schmidt\.stargatewissen@gmail\.com$
- ^fomafix@googlemail\.com$
- ^framawiki@tools\.wmflabs\.org$
+   - ^gerritpatchuploader@gmail\.com$
- ^geofbot@gmail\.com$
- ^glaisher\.wiki@gmail\.com$
- ^hartman\.wiki@gmail\.com$

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I91116f694aba89bfca4569f2c500028c49b785cb
Gerrit-PatchSet: 2
Gerrit-Project: integration/config
Gerrit-Branch: master
Gerrit-Owner: Zoranzoki21 

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

[MediaWiki-commits] [Gerrit] mediawiki...MinervaNeue[master]: Unify SVG markup

2017-11-04 Thread VolkerE (Code Review)

VolkerE has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389290 )

Change subject: Unify SVG markup
..

Unify SVG markup

Align SVG markup across Foundation products by:
- unifying XML declaration,
- adding `width` and `height` attributes where missing,
- removing all unnecessary `g` group elements where applicable,
- putting attributes in order,
- removing unnecessary ids and
- unifying whitespace.

Bug: T178867
Change-Id: I6860c9259553e0e41f7656f9e98534db8e02bbc2
---
M resources/skins.minerva.base.styles/magnifying-glass.svg
M resources/skins.minerva.content.styles/images/external-link-ltr-icon.svg
M resources/skins.minerva.content.styles/images/external-link-rtl-icon.svg
M resources/skins.minerva.icons.images.scripts/download.svg
M resources/skins.minerva.icons.images.scripts/userAnonymous.svg
M resources/skins.minerva.icons.images.scripts/watch.svg
M resources/skins.minerva.icons.images.scripts/watched.svg
M resources/skins.minerva.icons.images.variants/arrow.svg
M resources/skins.minerva.icons.images/bell.svg
M resources/skins.minerva.icons.images/edit.svg
M resources/skins.minerva.icons.images/editLocked.svg
M resources/skins.minerva.icons.images/languageSwitcher.svg
M resources/skins.minerva.mainMenu.icons/anonymous.svg
M resources/skins.minerva.mainMenu.icons/contributions.svg
M resources/skins.minerva.mainMenu.icons/home.svg
M resources/skins.minerva.mainMenu.icons/logout.svg
M resources/skins.minerva.mainMenu.icons/nearby.svg
M resources/skins.minerva.mainMenu.icons/profile.svg
M resources/skins.minerva.mainMenu.icons/random.svg
M resources/skins.minerva.mainMenu.icons/settings.svg
M resources/skins.minerva.mainMenu.icons/watchlist.svg
M skinStyles/mobile.startup/images/error.svg
22 files changed, 22 insertions(+), 22 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/skins/MinervaNeue 
refs/changes/90/389290/1

diff --git a/resources/skins.minerva.base.styles/magnifying-glass.svg 
b/resources/skins.minerva.base.styles/magnifying-glass.svg
index 7103adb..42f4289 100644
--- a/resources/skins.minerva.base.styles/magnifying-glass.svg
+++ b/resources/skins.minerva.base.styles/magnifying-glass.svg
@@ -1 +1 @@
-http://www.w3.org/2000/svg"; 
viewBox="0 0 24 24">
\ No newline at end of file
+http://www.w3.org/2000/svg"; 
width="24" height="24" viewBox="0 0 24 24">
\ No newline at end of file
diff --git 
a/resources/skins.minerva.content.styles/images/external-link-ltr-icon.svg 
b/resources/skins.minerva.content.styles/images/external-link-ltr-icon.svg
index 499dc22..0f4e407 100644
--- a/resources/skins.minerva.content.styles/images/external-link-ltr-icon.svg
+++ b/resources/skins.minerva.content.styles/images/external-link-ltr-icon.svg
@@ -1 +1 @@
-http://www.w3.org/2000/svg"; 
width="10" height="10">
\ No newline at end of file
+http://www.w3.org/2000/svg"; 
width="10" height="10" viewBox="0 0 10 10">
\ No newline at end of file
diff --git 
a/resources/skins.minerva.content.styles/images/external-link-rtl-icon.svg 
b/resources/skins.minerva.content.styles/images/external-link-rtl-icon.svg
index 2577607..061e385 100644
--- a/resources/skins.minerva.content.styles/images/external-link-rtl-icon.svg
+++ b/resources/skins.minerva.content.styles/images/external-link-rtl-icon.svg
@@ -1 +1 @@
-http://www.w3.org/2000/svg"; 
width="10" height="10">
\ No newline at end of file
+http://www.w3.org/2000/svg"; 
width="10" height="10" viewBox="0 0 10 10">
\ No newline at end of file
diff --git a/resources/skins.minerva.icons.images.scripts/download.svg 
b/resources/skins.minerva.icons.images.scripts/download.svg
index 1a19a9f..18b2388 100644
--- a/resources/skins.minerva.icons.images.scripts/download.svg
+++ b/resources/skins.minerva.icons.images.scripts/download.svg
@@ -1 +1 @@
-http://www.w3.org/2000/svg";>
\ No newline at end of file
+http://www.w3.org/2000/svg"; 
width="24" height="24" viewBox="0 0 24 24">
\ No newline at end of file
diff --git a/resources/skins.minerva.icons.images.scripts/userAnonymous.svg 
b/resources/skins.minerva.icons.images.scripts/userAnonymous.svg
index cccd4de..17ed037 100644
--- a/resources/skins.minerva.icons.images.scripts/userAnonymous.svg
+++ b/resources/skins.minerva.icons.images.scripts/userAnonymous.svg
@@ -1 +1 @@
-http://www.w3.org/2000/svg"; 
viewBox="0 0 16 16">
\ No newline at end of file
+http://www.w3.org/2000/svg"; 
width="16" height="16" viewBox="0 0 16 16">
\ No newline at end of file
diff --git a/resources/skins.minerva.icons.images.scripts/watch.svg 
b/resources/skins.minerva.icons.images.scripts/watch.svg
index ed9baa9..9a4a99d 100644
--- a/resources/skins.minerva.icons.images.scripts/watch.svg
+++ b/resources/skins.minerva.icons.images.scripts/watch.svg
@@ -1 +1 @@
-http://www.w3.org/2000/svg";>
\ No newline at end of file
+http://www.w3.org/2000/svg"; 
width="24" height="24" viewBox="0 0 24 24">
\ No newline at end of file
diff --git a/resources/skins.minerva

[MediaWiki-commits] [Gerrit] mediawiki...Renameuser[master]: Use proper Maintenance class accessors

2017-11-04 Thread MaxSem (Code Review)

MaxSem has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389289 )

Change subject: Use proper Maintenance class accessors
..

Use proper Maintenance class accessors

Change-Id: I424197924afd316fc16d427ffc1aa9bd9b5caffb
---
M maintenance/renameUserCleanup.php
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Renameuser 
refs/changes/89/389289/1

diff --git a/maintenance/renameUserCleanup.php 
b/maintenance/renameUserCleanup.php
index a1ec1d7..d13f0c0 100644
--- a/maintenance/renameUserCleanup.php
+++ b/maintenance/renameUserCleanup.php
@@ -32,12 +32,12 @@
 class RenameUserCleanup extends Maintenance {
public function __construct() {
parent::__construct();
-   $this->mDescription = 'Maintenance script to finish incomplete 
rename user,'
-   . ' in particular to reassign edits that were missed';
+   $this->addDescription( 'Maintenance script to finish incomplete 
rename user,'
+   . ' in particular to reassign edits that were missed' );
$this->addOption( 'olduser', 'Old user name', true, true );
$this->addOption( 'newuser', 'New user name', true, true );
$this->addOption( 'olduid', 'Old user id in revision records 
(DANGEROUS)', false, true );
-   $this->mBatchSize = 1000;
+   $this->setBatchSize( 1000 );
}
 
public function execute() {

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I424197924afd316fc16d427ffc1aa9bd9b5caffb
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Renameuser
Gerrit-Branch: master
Gerrit-Owner: MaxSem 

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

[MediaWiki-commits] [Gerrit] mediawiki...VisualEditor[REL1_29]: Follow-up I488fd7e74: Adjust saveFail calls for new function...

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Follow-up I488fd7e74: Adjust saveFail calls for new function 
signature
..


Follow-up I488fd7e74: Adjust saveFail calls for new function signature

Bug: T164157
Change-Id: I41b946af5a8b80211b0672d7012835f2c9e1dc2b
(cherry picked from commit 998ae22b2de690fe18d49489e3e90aa33267731b)
---
M modules/ve-mw/init/ve.init.mw.ArticleTarget.js
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Bartosz Dziewoński: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/ve-mw/init/ve.init.mw.ArticleTarget.js 
b/modules/ve-mw/init/ve.init.mw.ArticleTarget.js
index e033b48..7a66419 100644
--- a/modules/ve-mw/init/ve.init.mw.ArticleTarget.js
+++ b/modules/ve-mw/init/ve.init.mw.ArticleTarget.js
@@ -503,12 +503,12 @@
var data = response.visualeditoredit;
this.saving = false;
if ( !data ) {
-   this.saveFail( doc, saveData, null, 'Invalid response from 
server', response );
+   this.saveFail( doc, saveData, false, null, 'Invalid response 
from server', response );
} else if ( data.result !== 'success' ) {
// Note, this could be any of db failure, hookabort, badtoken 
or even a captcha
-   this.saveFail( doc, saveData, null, 'Save failure', response );
+   this.saveFail( doc, saveData, false, null, 'Save failure', 
response );
} else if ( typeof data.content !== 'string' ) {
-   this.saveFail( doc, saveData, null, 'Invalid HTML content in 
response from server', response );
+   this.saveFail( doc, saveData, false, null, 'Invalid HTML 
content in response from server', response );
} else {
this.saveComplete(
data.content,

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I41b946af5a8b80211b0672d7012835f2c9e1dc2b
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: REL1_29
Gerrit-Owner: MacFan4000 
Gerrit-Reviewer: Bartosz Dziewoński 
Gerrit-Reviewer: Esanders 
Gerrit-Reviewer: Jforrester 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] pywikibot/core[master]: noreferences.py: Localise name of template reflist on serbia...

2017-11-04 Thread Zoranzoki21 (Code Review)

Zoranzoki21 has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389288 )

Change subject: noreferences.py: Localise name of template reflist on serbian 
language
..

noreferences.py: Localise name of template reflist on serbian language

Bug: T176019
Change-Id: Idddcc117495684f4cd7efd9c8f24b7b6f9e67501
---
M scripts/noreferences.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/pywikibot/core 
refs/changes/88/389288/1

diff --git a/scripts/noreferences.py b/scripts/noreferences.py
index b0b5b7d..7f0a213 100755
--- a/scripts/noreferences.py
+++ b/scripts/noreferences.py
@@ -418,7 +418,7 @@
 'ru': [u'Reflist', u'Ref-list', u'Refs', u'Sources',
u'Примечания', u'Список примечаний',
u'Сноска', u'Сноски'],
-'sr': ['Reflist'],
+'sr': ['Reflist', 'Референце', 'Извори'],
 'szl': [u'Przipisy', u'Připisy'],
 'th': [u'รายการอ้างอิง'],
 'zh': [u'Reflist', u'RefFoot', u'NoteFoot'],

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idddcc117495684f4cd7efd9c8f24b7b6f9e67501
Gerrit-PatchSet: 1
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: Zoranzoki21 

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

[MediaWiki-commits] [Gerrit] mediawiki...deploy[CELERY_4]: Bump ores to get new subtask design

2017-11-04 Thread Awight (Code Review)

Awight has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/389287 )

Change subject: Bump ores to get new subtask design
..


Bump ores to get new subtask design

Bug: T179524
Change-Id: I889ea04d3559fc0d614082f81250e9d958bcd174
---
M submodules/ores
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Awight: Verified; Looks good to me, approved



diff --git a/submodules/ores b/submodules/ores
index b3c7961..1823595 16
--- a/submodules/ores
+++ b/submodules/ores
@@ -1 +1 @@
-Subproject commit b3c7961c3b0a725126e45cd65922cc60f5a13b51
+Subproject commit 182359573bdeb359598ab33b3be7535ef0556d8d

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I889ea04d3559fc0d614082f81250e9d958bcd174
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/ores/deploy
Gerrit-Branch: CELERY_4
Gerrit-Owner: Awight 
Gerrit-Reviewer: Awight 

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

[MediaWiki-commits] [Gerrit] mediawiki...deploy[CELERY_4]: Use branched ores to continue Celery 4 work

2017-11-04 Thread Awight (Code Review)

Awight has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/388160 )

Change subject: Use branched ores to continue Celery 4 work
..


Use branched ores to continue Celery 4 work

Bug: T178441
Change-Id: If2579e5db7d3159dbd7f1292e2b5df21313f1c9a
---
M .gitreview
M submodules/ores
2 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Zppix: Looks good to me, but someone else must approve
  Halfak: Verified; Looks good to me, approved



diff --git a/.gitreview b/.gitreview
index 9d91f0d..3c6d0ca 100644
--- a/.gitreview
+++ b/.gitreview
@@ -2,5 +2,5 @@
 host=gerrit.wikimedia.org
 port=29418
 project=mediawiki/services/ores/deploy.git
-defaultbranch=master
+defaultbranch=CELERY_4
 defaultrebase=0
diff --git a/submodules/ores b/submodules/ores
index a65a154..b3c7961 16
--- a/submodules/ores
+++ b/submodules/ores
@@ -1 +1 @@
-Subproject commit a65a154e41340f3fcd5018b48423207888e8424e
+Subproject commit b3c7961c3b0a725126e45cd65922cc60f5a13b51

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

Gerrit-MessageType: merged
Gerrit-Change-Id: If2579e5db7d3159dbd7f1292e2b5df21313f1c9a
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/ores/deploy
Gerrit-Branch: CELERY_4
Gerrit-Owner: Awight 
Gerrit-Reviewer: Awight 
Gerrit-Reviewer: Halfak 
Gerrit-Reviewer: Zppix 

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

[MediaWiki-commits] [Gerrit] mediawiki...deploy[CELERY_4]: Bump ores to get new subtask design

2017-11-04 Thread Awight (Code Review)

Awight has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389287 )

Change subject: Bump ores to get new subtask design
..

Bump ores to get new subtask design

Bug: T179524
Change-Id: I889ea04d3559fc0d614082f81250e9d958bcd174
---
M submodules/ores
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/services/ores/deploy 
refs/changes/87/389287/1

diff --git a/submodules/ores b/submodules/ores
index b3c7961..1823595 16
--- a/submodules/ores
+++ b/submodules/ores
@@ -1 +1 @@
-Subproject commit b3c7961c3b0a725126e45cd65922cc60f5a13b51
+Subproject commit 182359573bdeb359598ab33b3be7535ef0556d8d

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I889ea04d3559fc0d614082f81250e9d958bcd174
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/ores/deploy
Gerrit-Branch: CELERY_4
Gerrit-Owner: Awight 

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

[MediaWiki-commits] [Gerrit] mediawiki...BlueSpiceEditNotifyConnector[master]: build: Adding MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/388293 )

Change subject: build: Adding MinusX
..


build: Adding MinusX

Change-Id: I2945d4d56da7a07e6cf44dd7203f909014dd082f
---
M composer.json
1 file changed, 7 insertions(+), 2 deletions(-)

Approvals:
  Umherirrender: Verified
  Legoktm: Looks good to me, approved



diff --git a/composer.json b/composer.json
index 73f0cf1..d119b9c 100644
--- a/composer.json
+++ b/composer.json
@@ -9,11 +9,16 @@
"require-dev": {
"jakub-onderka/php-parallel-lint": "0.9.2",
"jakub-onderka/php-console-highlighter": "0.3.2",
-   "phpunit/phpunit": "4.8.*"
+   "phpunit/phpunit": "4.8.*",
+   "mediawiki/minus-x": "0.2.0"
},
"scripts": {
"test": [
-   "parallel-lint . --exclude vendor --exclude 
node_modules"
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
]
}
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2945d4d56da7a07e6cf44dd7203f909014dd082f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceEditNotifyConnector
Gerrit-Branch: master
Gerrit-Owner: Legoktm 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: Ljonka 
Gerrit-Reviewer: Mglaser 
Gerrit-Reviewer: Pwirth 
Gerrit-Reviewer: Robert Vogel 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: NamespaceInputWidget: Fix type documentation

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: NamespaceInputWidget: Fix type documentation
..


NamespaceInputWidget: Fix type documentation

Change-Id: Ifc8e8be7bb7423d581f62427a4e48d70d73c714b
---
M includes/widget/NamespaceInputWidget.php
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/includes/widget/NamespaceInputWidget.php 
b/includes/widget/NamespaceInputWidget.php
index 3e86738..3e913b0 100644
--- a/includes/widget/NamespaceInputWidget.php
+++ b/includes/widget/NamespaceInputWidget.php
@@ -18,7 +18,7 @@
 * @param array $config Configuration options
 * @param string $config['includeAllValue'] If specified, add a "all 
namespaces" option to the
 * namespace dropdown, and use this as the input value for it
-* @param number[] $config['exclude'] List of namespace numbers to 
exclude from the selector
+* @param int[] $config['exclude'] List of namespace numbers to exclude 
from the selector
 */
public function __construct( array $config = [] ) {
// Configuration initialization

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifc8e8be7bb7423d581f62427a4e48d70d73c714b
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: MaxSem 
Gerrit-Reviewer: Florianschmidtwelzow 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Don't globally backlist PhanRedefineFunctionInternal

2017-11-04 Thread MaxSem (Code Review)

MaxSem has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389286 )

Change subject: Don't globally backlist PhanRedefineFunctionInternal
..

Don't globally backlist PhanRedefineFunctionInternal

Instead, suppress the only occurence that we have to keep for now.

Change-Id: I050a8dcb2efc92f4f17c8b8544686310ad6cadc7
---
M includes/GlobalFunctions.php
M tests/phan/config.php
2 files changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/86/389286/1

diff --git a/includes/GlobalFunctions.php b/includes/GlobalFunctions.php
index 1cff881..389817b 100644
--- a/includes/GlobalFunctions.php
+++ b/includes/GlobalFunctions.php
@@ -67,6 +67,8 @@
 * Longer explanation: http://www.emerose.com/timing-attacks-explained
 *
 * @codeCoverageIgnore
+* @suppress PhanRedefineFunctionInternal
+*
 * @param string $known_string Fixed-length secret string to compare 
against
 * @param string $user_string User-provided string
 * @return bool True if the strings are the same, false otherwise
diff --git a/tests/phan/config.php b/tests/phan/config.php
index 0df8357..14cdbad 100644
--- a/tests/phan/config.php
+++ b/tests/phan/config.php
@@ -323,8 +323,6 @@
"PhanParamTooManyCallable",
// approximate error count: 3
"PhanParamTooManyInternal",
-   // approximate error count: 1
-   "PhanRedefineFunctionInternal",
// approximate error count: 2
"PhanTraitParentReference",
// approximate error count: 3

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I050a8dcb2efc92f4f17c8b8544686310ad6cadc7
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: MaxSem 

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

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Fix annotation

2017-11-04 Thread MaxSem (Code Review)

MaxSem has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389284 )

Change subject: Fix annotation
..

Fix annotation

Change-Id: Ifc8e8be7bb7423d581f62427a4e48d70d73c714b
---
M includes/widget/NamespaceInputWidget.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/84/389284/1

diff --git a/includes/widget/NamespaceInputWidget.php 
b/includes/widget/NamespaceInputWidget.php
index 3e86738..3e913b0 100644
--- a/includes/widget/NamespaceInputWidget.php
+++ b/includes/widget/NamespaceInputWidget.php
@@ -18,7 +18,7 @@
 * @param array $config Configuration options
 * @param string $config['includeAllValue'] If specified, add a "all 
namespaces" option to the
 * namespace dropdown, and use this as the input value for it
-* @param number[] $config['exclude'] List of namespace numbers to 
exclude from the selector
+* @param int[] $config['exclude'] List of namespace numbers to exclude 
from the selector
 */
public function __construct( array $config = [] ) {
// Configuration initialization

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifc8e8be7bb7423d581f62427a4e48d70d73c714b
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: MaxSem 

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

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Update for Phan 0.10.1

2017-11-04 Thread MaxSem (Code Review)

MaxSem has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389285 )

Change subject: Update for Phan 0.10.1
..

Update for Phan 0.10.1

Change-Id: Id292944b80eb0c81956b203ce95bb0def3b4b0a3
---
M tests/phan/config.php
1 file changed, 36 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/85/389285/1

diff --git a/tests/phan/config.php b/tests/phan/config.php
index ccc0177..0df8357 100644
--- a/tests/phan/config.php
+++ b/tests/phan/config.php
@@ -295,20 +295,32 @@
 * to this black-list to inhibit them from being reported.
 */
'suppress_issue_types' => [
+   // approximate error count: 1
+   "PhanAccessMethodProtected",
+   // approximate error count: 29
+   "PhanCommentParamOnEmptyParamList",
+   // approximate error count: 33
+   "PhanCommentParamWithoutRealParam",
// approximate error count: 8
"PhanDeprecatedClass",
// approximate error count: 415
"PhanDeprecatedFunction",
// approximate error count: 25
"PhanDeprecatedProperty",
+   // approximate error count: 17
+   "PhanNonClassMethodCall",
// approximate error count: 11
"PhanParamReqAfterOpt",
// approximate error count: 888
"PhanParamSignatureMismatch",
// approximate error count: 7
"PhanParamSignatureMismatchInternal",
+   // approximate error count: 1
+   "PhanParamSignatureRealMismatchTooFewParameters",
// approximate error count: 125
"PhanParamTooMany",
+   // approximate error count: 1
+   "PhanParamTooManyCallable",
// approximate error count: 3
"PhanParamTooManyInternal",
// approximate error count: 1
@@ -317,12 +329,28 @@
"PhanTraitParentReference",
// approximate error count: 3
"PhanTypeComparisonFromArray",
+   // approximate error count: 2
+   "PhanTypeComparisonToArray",
// approximate error count: 3
"PhanTypeInvalidRightOperand",
+   // approximate error count: 1
+   "PhanTypeMagicVoidWithReturn",
// approximate error count: 218
"PhanTypeMismatchArgument",
// approximate error count: 13
"PhanTypeMismatchArgumentInternal",
+   // approximate error count: 6
+   "PhanTypeMismatchDeclaredParam",
+   // approximate error count: 111
+   "PhanTypeMismatchDeclaredParamNullable",
+   // approximate error count: 1
+   "PhanTypeMismatchDefault",
+   // approximate error count: 5
+   "PhanTypeMismatchDimAssignment",
+   // approximate error count: 2
+   "PhanTypeMismatchDimEmpty",
+   // approximate error count: 1
+   "PhanTypeMismatchDimFetch",
// approximate error count: 14
"PhanTypeMismatchForeach",
// approximate error count: 56
@@ -333,6 +361,8 @@
"PhanTypeMissingReturn",
// approximate error count: 5
"PhanTypeNonVarPassByRef",
+   // approximate error count: 1
+   "PhanUndeclaredClassInCallable",
// approximate error count: 32
"PhanUndeclaredConstant",
// approximate error count: 233
@@ -341,6 +371,12 @@
"PhanUndeclaredProperty",
// approximate error count: 3
"PhanUndeclaredStaticMethod",
+   // approximate error count: 11
+   "PhanUndeclaredTypeReturnType",
+   // approximate error count: 27
+   "PhanUndeclaredVariable",
+   // approximate error count: 58
+   "PhanUndeclaredVariableDim",
],
 
/**

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id292944b80eb0c81956b203ce95bb0def3b4b0a3
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: MaxSem 

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

[MediaWiki-commits] [Gerrit] mediawiki...Tabber[master]: Added use of HTML5 push state with a fallback for older brow...

2017-11-04 Thread Marcus-crane (Code Review)

Marcus-crane has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389283 )

Change subject: Added use of HTML5 push state with a fallback for older 
browsers.
..

Added use of HTML5 push state with a fallback for older browsers.

Use of location.hash when a tab name is identical to a heading, will
cause the browser to jump to the respective heading via anchor tag.

One effect of using push state is that this bug is then mitigated in
modern browsers.

Change-Id: I51bc0c244b5be0f55493a0ee2e233c9e0cbd6552
---
M js/tabber.js
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Tabber 
refs/changes/83/389283/1

diff --git a/js/tabber.js b/js/tabber.js
index 52bf8d3..9658039 100644
--- a/js/tabber.js
+++ b/js/tabber.js
@@ -40,7 +40,11 @@
nav.on( 'click', 'a', function( e ) {
var title = $( this ).attr( 'title' );
e.preventDefault();
-   location.hash = '#' + title;
+   if (history.pushState) {
+   history.pushState(null, null, '#' + 
title);
+   } else {
+   location.hash = '#' + title;
+   }
showContent( title );
} );
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I51bc0c244b5be0f55493a0ee2e233c9e0cbd6552
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Tabber
Gerrit-Branch: master
Gerrit-Owner: Marcus-crane 

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

[MediaWiki-commits] [Gerrit] mediawiki...GitHub[master]: build: Adding MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/388674 )

Change subject: build: Adding MinusX
..


build: Adding MinusX

Change-Id: If42cf10f09ea6118c71e8a482a0b3886720d4ba3
---
M composer.json
1 file changed, 8 insertions(+), 3 deletions(-)

Approvals:
  Umherirrender: Verified
  Legoktm: Looks good to me, approved



diff --git a/composer.json b/composer.json
index 4482727..9ee70d3 100644
--- a/composer.json
+++ b/composer.json
@@ -28,7 +28,7 @@
"psr-4": {
"GitHub\\": "src/"
},
-   "files" : [
+   "files": [
"GitHub.php"
],
"classmap": [
@@ -45,11 +45,16 @@
},
"require-dev": {
"jakub-onderka/php-parallel-lint": "0.9.2",
-   "jakub-onderka/php-console-highlighter": "0.3.2"
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
},
"scripts": {
"test": [
-   "parallel-lint . --exclude vendor"
+   "parallel-lint . --exclude vendor",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
]
}
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: If42cf10f09ea6118c71e8a482a0b3886720d4ba3
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/GitHub
Gerrit-Branch: master
Gerrit-Owner: Legoktm 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...PubSubHubbub[master]: build: Adding MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/388797 )

Change subject: build: Adding MinusX
..


build: Adding MinusX

Change-Id: Ie26925dbf3721d104a373556465909ea98b346e0
---
M composer.json
1 file changed, 14 insertions(+), 3 deletions(-)

Approvals:
  Umherirrender: Verified
  Legoktm: Looks good to me, approved



diff --git a/composer.json b/composer.json
index 8928ac2..736c2d6 100644
--- a/composer.json
+++ b/composer.json
@@ -2,7 +2,13 @@
"name": "bp2013n2/pubsubhubbub",
"type": "mediawiki-extension",
"description": "Allows publishing changes to a PubSubHubbub hub",
-   "keywords": ["pubsubhubbub", "push", "rss", "wikibase", "wikidata"],
+   "keywords": [
+   "pubsubhubbub",
+   "push",
+   "rss",
+   "wikibase",
+   "wikidata"
+   ],
"homepage": "https://www.mediawiki.org/wiki/Extension:PubSubHubbub";,
"license": "GPL-2.0+",
"authors": [
@@ -36,11 +42,16 @@
},
"require-dev": {
"jakub-onderka/php-parallel-lint": "0.9.2",
-   "jakub-onderka/php-console-highlighter": "0.3.2"
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
},
"scripts": {
"test": [
-   "parallel-lint . --exclude vendor"
+   "parallel-lint . --exclude vendor",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
]
}
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie26925dbf3721d104a373556465909ea98b346e0
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/PubSubHubbub
Gerrit-Branch: master
Gerrit-Owner: Legoktm 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Revert #contentSub and #jump-to-nav margin changes

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Revert #contentSub and #jump-to-nav margin changes
..


Revert #contentSub and #jump-to-nav margin changes

The reduced space between the subtitle and the article content
is apparently not desirable.

* Revert "Remove contentSub from layout when empty"
  This reverts commit 09e3ac0c3a816b77d041c249ccd5b8bb88ba8ef5.
* Revert "Fix margin of #jump-to-nav"
  This reverts commit 29b9ee20207b887006f38c85464f6019dae85b69.

Bug: T178626
Change-Id: I84c89053752ab2dc3595b114cf7970860467
---
M resources/src/mediawiki.skinning/interface.css
1 file changed, 0 insertions(+), 11 deletions(-)

Approvals:
  Bartosz Dziewoński: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/resources/src/mediawiki.skinning/interface.css 
b/resources/src/mediawiki.skinning/interface.css
index bf92086..3e0d2b9 100644
--- a/resources/src/mediawiki.skinning/interface.css
+++ b/resources/src/mediawiki.skinning/interface.css
@@ -76,17 +76,6 @@
width: auto;
 }
 
-#contentSub:empty,
-#contentSub2:empty {
-   display: none;
-}
-
-#contentSub:empty ~ #jump-to-nav,
-#contentSub2:empty ~ #jump-to-nav {
-   margin-top: 0;
-   margin-bottom: 0;
-}
-
 span.subpages {
display: block;
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I84c89053752ab2dc3595b114cf7970860467
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Bartosz Dziewoński 
Gerrit-Reviewer: Bartosz Dziewoński 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...SemanticHighcharts[master]: build: Adding MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/388820 )

Change subject: build: Adding MinusX
..


build: Adding MinusX

Change-Id: Id2b9467ee5e0df5243ad7bab53403c98b871e7cf
---
M composer.json
1 file changed, 9 insertions(+), 4 deletions(-)

Approvals:
  Umherirrender: Verified
  Legoktm: Looks good to me, approved



diff --git a/composer.json b/composer.json
index d26fa8b..11ffcc4 100644
--- a/composer.json
+++ b/composer.json
@@ -18,20 +18,25 @@
}
],
"autoload": {
-   "files" : [
+   "files": [
"SemanticHighcharts.php"
],
-   "classmap":[
+   "classmap": [
"src/"
]
},
"require-dev": {
"jakub-onderka/php-parallel-lint": "0.9.2",
-   "jakub-onderka/php-console-highlighter": "0.3.2"
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
},
"scripts": {
"test": [
-   "parallel-lint . --exclude vendor --exclude 
node_modules --exclude extensions"
+   "parallel-lint . --exclude vendor --exclude 
node_modules --exclude extensions",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
]
}
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id2b9467ee5e0df5243ad7bab53403c98b871e7cf
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/SemanticHighcharts
Gerrit-Branch: master
Gerrit-Owner: Legoktm 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...SemanticHighcharts[master]: Require mediawiki/validator in composer.json

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/389281 )

Change subject: Require mediawiki/validator in composer.json
..


Require mediawiki/validator in composer.json

Change-Id: I1a384168efd5e8228487266ffdd2cdaa614177f0
---
M composer.json
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Umherirrender: Verified; Looks good to me, approved



diff --git a/composer.json b/composer.json
index 474f7a0..d26fa8b 100644
--- a/composer.json
+++ b/composer.json
@@ -6,6 +6,7 @@
"require": {
"php": ">=5.3.0",
"composer/installers": "1.*,>=1.0.1",
+   "mediawiki/validator": ">=1.0.0.1",
"mediawiki/semantic-media-wiki": ">=1.9",
"netbrain/highcharts-js": "3.0.9"
},

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1a384168efd5e8228487266ffdd2cdaa614177f0
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/SemanticHighcharts
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...SemanticHighcharts[master]: Require mediawiki/validator in composer.json

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389281 )

Change subject: Require mediawiki/validator in composer.json
..

Require mediawiki/validator in composer.json

Change-Id: I1a384168efd5e8228487266ffdd2cdaa614177f0
---
M composer.json
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/SemanticHighcharts 
refs/changes/81/389281/1

diff --git a/composer.json b/composer.json
index 474f7a0..d26fa8b 100644
--- a/composer.json
+++ b/composer.json
@@ -6,6 +6,7 @@
"require": {
"php": ">=5.3.0",
"composer/installers": "1.*,>=1.0.1",
+   "mediawiki/validator": ">=1.0.0.1",
"mediawiki/semantic-media-wiki": ">=1.9",
"netbrain/highcharts-js": "3.0.9"
},

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1a384168efd5e8228487266ffdd2cdaa614177f0
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/SemanticHighcharts
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Revert #contentSub and #jump-to-nav margin changes

2017-11-04 Thread Code Review

Bartosz Dziewoński has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389282 )

Change subject: Revert #contentSub and #jump-to-nav margin changes
..

Revert #contentSub and #jump-to-nav margin changes

The reduced space between the subtitle and the article content
is apparently not desirable.

* Revert "Remove contentSub from layout when empty"
  This reverts commit 09e3ac0c3a816b77d041c249ccd5b8bb88ba8ef5.
* Revert "Fix margin of #jump-to-nav"
  This reverts commit 29b9ee20207b887006f38c85464f6019dae85b69.

Bug: T178626
Change-Id: I84c89053752ab2dc3595b114cf7970860467
---
M resources/src/mediawiki.skinning/interface.css
1 file changed, 0 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/82/389282/1

diff --git a/resources/src/mediawiki.skinning/interface.css 
b/resources/src/mediawiki.skinning/interface.css
index bf92086..3e0d2b9 100644
--- a/resources/src/mediawiki.skinning/interface.css
+++ b/resources/src/mediawiki.skinning/interface.css
@@ -76,17 +76,6 @@
width: auto;
 }
 
-#contentSub:empty,
-#contentSub2:empty {
-   display: none;
-}
-
-#contentSub:empty ~ #jump-to-nav,
-#contentSub2:empty ~ #jump-to-nav {
-   margin-top: 0;
-   margin-bottom: 0;
-}
-
 span.subpages {
display: block;
 }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I84c89053752ab2dc3595b114cf7970860467
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Bartosz Dziewoński 

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

[MediaWiki-commits] [Gerrit] mediawiki...Wikisource[master]: build: Allow empty input list for stylelint

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Allow empty input list for stylelint
..


build: Allow empty input list for stylelint

Change-Id: I1a07cfe2d090d17e3a762b08f59d69e7b759e113
---
M Gruntfile.js
1 file changed, 3 insertions(+), 0 deletions(-)

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



diff --git a/Gruntfile.js b/Gruntfile.js
index d65e184..afa2bd1 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -15,6 +15,9 @@
]
},
stylelint: {
+   options: {
+   allowEmptyInput: true
+   },
all: [
'modules/**/*.css'
]

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1a07cfe2d090d17e3a762b08f59d69e7b759e113
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikisource
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...Wikisource[master]: build: Adding MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Adding MinusX
..


build: Adding MinusX

Change-Id: Iaf7d724bd930b3ddd0085a26a234cebcee346f89
---
M composer.json
1 file changed, 6 insertions(+), 3 deletions(-)

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



diff --git a/composer.json b/composer.json
index 131c783..a811366 100644
--- a/composer.json
+++ b/composer.json
@@ -7,15 +7,18 @@
"require-dev": {
"jakub-onderka/php-parallel-lint": "0.9.2",
"mediawiki/mediawiki-codesniffer": "14.1.0",
-   "jakub-onderka/php-console-highlighter": "0.3.2"
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
},
"scripts": {
"test": [
"parallel-lint . --exclude vendor",
-   "phpcs -p -s"
+   "phpcs -p -s",
+   "minus-x check ."
],
"fix": [
-   "phpcbf"
+   "phpcbf",
+   "minus-x fix ."
]
}
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaf7d724bd930b3ddd0085a26a234cebcee346f89
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/Wikisource
Gerrit-Branch: master
Gerrit-Owner: Legoktm 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...Wikisource[master]: build: Allow empty input list for stylelint

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389280 )

Change subject: build: Allow empty input list for stylelint
..

build: Allow empty input list for stylelint

Change-Id: I1a07cfe2d090d17e3a762b08f59d69e7b759e113
---
M Gruntfile.js
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikisource 
refs/changes/80/389280/1

diff --git a/Gruntfile.js b/Gruntfile.js
index d65e184..afa2bd1 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -15,6 +15,9 @@
]
},
stylelint: {
+   options: {
+   allowEmptyInput: true
+   },
all: [
'modules/**/*.css'
]

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1a07cfe2d090d17e3a762b08f59d69e7b759e113
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikisource
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...TranslateSvg[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/389192 )

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: Ib954c800895f640333d1e1c0fd04a88eb96dff68
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

Approvals:
  Umherirrender: Verified; Looks good to me, approved



diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib954c800895f640333d1e1c0fd04a88eb96dff68
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/TranslateSvg
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...PhpTagsStorage[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/389088 )

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I8d8f76edf1a685276254e44a9c53edb3bf021b8e
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

Approvals:
  Umherirrender: Verified; Looks good to me, approved



diff --git a/.gitignore b/.gitignore
index c369f1e..8aa2a07 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 # Editors
 *.kate-swp
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8d8f76edf1a685276254e44a9c53edb3bf021b8e
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/PhpTagsStorage
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...PhpTagsStorage[master]: Remove mw-parser-output wrapper class

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/389264 )

Change subject: Remove mw-parser-output wrapper class
..


Remove mw-parser-output wrapper class

This extensions parse small pieces of text,
which gets part of a bigger parsed text.
Remove the  around each small piece.

Found by failing unit tests:
2) StorageWikiPageTest::test_template_StorageTag
Page 'Test template DumpTags'
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'TAGS: one, two, three.
-'

Change-Id: If74760c4d4c7f949a53b10af54c9e995440883dd
---
M tests/phpunit/StorageWikiPage_Test.php
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Umherirrender: Verified; Looks good to me, approved



diff --git a/tests/phpunit/StorageWikiPage_Test.php 
b/tests/phpunit/StorageWikiPage_Test.php
index a2567e5..6bb4f25 100644
--- a/tests/phpunit/StorageWikiPage_Test.php
+++ b/tests/phpunit/StorageWikiPage_Test.php
@@ -154,6 +154,9 @@
 
$options = new ParserOptions();
$options->enableLimitReport( false );
+   if ( is_callable( [ $options, 'setWrapOutputClass' ] ) ) { // 
since 1.30
+   $options->setWrapOutputClass( false );
+   }
 
$output = $page->getContent()-> getParserOutput( 
$page->getTitle(), null, $options );
$this->assertEquals($output->getText(), "TAGS: one, two, 
three.\n", "Page 'Test template DumpTags'" );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: If74760c4d4c7f949a53b10af54c9e995440883dd
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/PhpTagsStorage
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...PhpTagsWidgets[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/389089 )

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I435c514db4e0168df0e16d691c34fde23b4efcbb
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

Approvals:
  Umherirrender: Verified; Looks good to me, approved



diff --git a/.gitignore b/.gitignore
index 62c8f71..5b91da5 100644
--- a/.gitignore
+++ b/.gitignore
@@ -2,6 +2,7 @@
 
 node_modules/
 vendor/
+composer.lock
 
 # Editors
 *.kate-swp
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I435c514db4e0168df0e16d691c34fde23b4efcbb
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/PhpTagsWidgets
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...PhpTagsStorage[master]: Remove mw-parser-output wrapper class

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389264 )

Change subject: Remove mw-parser-output wrapper class
..

Remove mw-parser-output wrapper class

This extensions parse small pieces of text,
which gets part of a bigger parsed text.
Remove the  around each small piece.

Found by failing unit tests:
2) StorageWikiPageTest::test_template_StorageTag
Page 'Test template DumpTags'
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'TAGS: one, two, three.
-'

Change-Id: If74760c4d4c7f949a53b10af54c9e995440883dd
---
M tests/phpunit/StorageWikiPage_Test.php
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/PhpTagsStorage 
refs/changes/64/389264/1

diff --git a/tests/phpunit/StorageWikiPage_Test.php 
b/tests/phpunit/StorageWikiPage_Test.php
index a2567e5..6bb4f25 100644
--- a/tests/phpunit/StorageWikiPage_Test.php
+++ b/tests/phpunit/StorageWikiPage_Test.php
@@ -154,6 +154,9 @@
 
$options = new ParserOptions();
$options->enableLimitReport( false );
+   if ( is_callable( [ $options, 'setWrapOutputClass' ] ) ) { // 
since 1.30
+   $options->setWrapOutputClass( false );
+   }
 
$output = $page->getContent()-> getParserOutput( 
$page->getTitle(), null, $options );
$this->assertEquals($output->getText(), "TAGS: one, two, 
three.\n", "Page 'Test template DumpTags'" );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If74760c4d4c7f949a53b10af54c9e995440883dd
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/PhpTagsStorage
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...Refreshed[master]: Split classes into their own files and folder

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Split classes into their own files and folder
..


Split classes into their own files and folder

Classes are now their own file, as well as in the folder `/includes`.

Change-Id: I847984a78bc3d2f31fffed10e31d926acdc1a9fe
---
R includes/RefreshedTemplate.php
A includes/SkinRefreshed.php
M skin.json
3 files changed, 63 insertions(+), 62 deletions(-)

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



diff --git a/Refreshed.skin.php b/includes/RefreshedTemplate.php
old mode 100755
new mode 100644
similarity index 91%
rename from Refreshed.skin.php
rename to includes/RefreshedTemplate.php
index cf097e4..98dfcbb
--- a/Refreshed.skin.php
+++ b/includes/RefreshedTemplate.php
@@ -1,64 +1,5 @@
 addMeta( 'viewport', 'width=device-width' );
-
-   // prevent iOS from zooming out when the sidebar is opened
-   $out->addHeadItem( 'viewportforios',
-   Html::element( 'meta', array(
-   'name' => 'viewport',
-   'content' => 'width=device-width, 
initial-scale=1.0'
-   ) )
-   );
-
-   // Inject webfont loader CSS file inline here so that it'll 
work even for IE11
-   // Conditional comments aren't supported in IE10+ so we have no 
way of loading
-   // this just for IE, so better to have all font declarations 
here (and
-   // maybe one day we'll rename the file to just "fontloader.css" 
or something)
-   // Based on some quick-ish testing on 25 July 2016, it appears 
that font
-   // declarations need to be loaded before they're used so that 
they work
-   // under IE(11).
-   // See https://phabricator.wikimedia.org/T134653 for more info.
-   $out->addHeadItem( 'webfontfix',
-Html::element( 'link', array(
-   'href' => $wgLocalStylePath . 
'/Refreshed/refreshed/styles/screen/iefontfix.css',
-   'rel' => 'stylesheet'
-   ) )
-   );
-
-   // Add JavaScript via ResourceLoader
-   $out->addModules( 'skins.refreshed.js' );
-   }
-
-   function setupSkinUserCss( OutputPage $out ) {
-   parent::setupSkinUserCss( $out );
-
-   // Add CSS via ResourceLoader
-   $out->addModuleStyles( array(
-   'mediawiki.skinning.interface',
-   'mediawiki.skinning.content.externallinks',
-   'skins.refreshed'
-   ) );
-   }
-}
-
 class RefreshedTemplate extends BaseTemplate {
/**
 * Parses MediaWiki:Refreshed-wiki-dropdown.
diff --git a/includes/SkinRefreshed.php b/includes/SkinRefreshed.php
new file mode 100644
index 000..b87db0b
--- /dev/null
+++ b/includes/SkinRefreshed.php
@@ -0,0 +1,60 @@
+addMeta( 'viewport', 'width=device-width' );
+
+   // prevent iOS from zooming out when the sidebar is opened
+   $out->addHeadItem( 'viewportforios',
+   Html::element( 'meta', array(
+   'name' => 'viewport',
+   'content' => 'width=device-width, 
initial-scale=1.0'
+   ) )
+   );
+
+   // Inject webfont loader CSS file inline here so that it'll 
work even for IE11
+   // Conditional comments aren't supported in IE10+ so we have no 
way of loading
+   // this just for IE, so better to have all font declarations 
here (and
+   // maybe one day we'll rename the file to just "fontloader.css" 
or something)
+   // Based on some quick-ish testing on 25 July 2016, it appears 
that font
+   // declarations need to be loaded before they're used so that 
they work
+   // under IE(11).
+   // See https://phabricator.wikimedia.org/T134653 for more info.
+   $out->addHeadItem( 'webfontfix',
+Html::element( 'link', array(
+   'href' => $wgLocalStylePath . 
'/Refreshed/refreshed/styles/screen/iefontfix.css',
+   'rel' => 'stylesheet'
+   ) )
+   );
+
+   // Add JavaScript via ResourceLoader
+   $out->addModules( 'skins.refreshed.js' );
+   }
+
+   function setupSkinUserCss( OutputPage $out ) {
+   parent::setupSkinUserCss( $out );
+
+   // Add CSS via ResourceLoader
+   $out->addModuleStyles( array(
+   'mediawiki.skinning.interface',
+   'mediawiki.skinning.content.externallinks',
+

[MediaWiki-commits] [Gerrit] mediawiki...Truglass[master]: Split classes into their own files and folder

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Split classes into their own files and folder
..


Split classes into their own files and folder

Classes are now their own file, as well as in the folder `/includes`.

Change-Id: Ib91efb1b49fa66dcc9c3b2d9b51b98d5bfb0e06e
---
A includes/SkinTruglass.php
R includes/TruglassTemplate.php
M skin.json
3 files changed, 35 insertions(+), 37 deletions(-)

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



diff --git a/includes/SkinTruglass.php b/includes/SkinTruglass.php
new file mode 100644
index 000..84ebeff
--- /dev/null
+++ b/includes/SkinTruglass.php
@@ -0,0 +1,32 @@
+truglassConfig = 
ConfigFactory::getDefaultInstance()->makeConfig( 'truglass' );
+   }
+
+   function setupSkinUserCss( OutputPage $out ) {
+   parent::setupSkinUserCss( $out );
+
+   // Add CSS
+   $out->addModuleStyles( 'skins.truglass' );
+   }
+
+   /**
+* Override to pass our Config instance to it
+*/
+   public function setupTemplate( $classname, $repository = false, 
$cache_dir = false ) {
+   return new $classname( $this->truglassConfig );
+   }
+}
diff --git a/Truglass.skin.php b/includes/TruglassTemplate.php
similarity index 93%
rename from Truglass.skin.php
rename to includes/TruglassTemplate.php
index a7e68e5..bd94b0b 100644
--- a/Truglass.skin.php
+++ b/includes/TruglassTemplate.php
@@ -12,40 +12,6 @@
  * @license http://www.gnu.org/copyleft/gpl.html GNU General Public License 
2.0 or later
  */
 
-/**
- * Inherit main code from SkinTemplate, set the CSS and template filter.
- * @ingroup Skins
- */
-class SkinTruglass extends SkinTemplate {
-   public $skinname = 'truglass', $stylename = 'truglass',
-   $template = 'TruglassTemplate', $useHeadElement = true;
-   /**
-* @var Config
-*/
-   private $truglassConfig;
-
-   public function __construct() {
-   $this->truglassConfig = 
ConfigFactory::getDefaultInstance()->makeConfig( 'truglass' );
-   }
-
-   function setupSkinUserCss( OutputPage $out ) {
-   parent::setupSkinUserCss( $out );
-
-   // Add CSS
-   $out->addModuleStyles( 'skins.truglass' );
-   }
-
-   /**
-* Override to pass our Config instance to it
-*/
-   public function setupTemplate( $classname, $repository = false, 
$cache_dir = false ) {
-   return new $classname( $this->truglassConfig );
-   }
-}
-
-/**
- * @todo document
- */
 class TruglassTemplate extends BaseTemplate {
 
/**
diff --git a/skin.json b/skin.json
index 0026da4..7e611e9 100644
--- a/skin.json
+++ b/skin.json
@@ -1,6 +1,6 @@
 {
"name": "Truglass",
-   "version": "4.1.6",
+   "version": "4.1.7",
"author": [
"Elliott Franklin Cable",
"Jack Phoenix",
@@ -22,8 +22,8 @@
]
},
"AutoloadClasses": {
-   "SkinTruglass": "Truglass.skin.php",
-   "TruglassTemplate": "Truglass.skin.php"
+   "SkinTruglass": "includes/SkinTruglass.php",
+   "TruglassTemplate": "includes/TruglassTemplate.php"
},
"ResourceFileModulePaths": {
"localBasePath": "",

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib91efb1b49fa66dcc9c3b2d9b51b98d5bfb0e06e
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/skins/Truglass
Gerrit-Branch: master
Gerrit-Owner: SamanthaNguyen 
Gerrit-Reviewer: Jack Phoenix 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...Gamepress[master]: Split classes into their own files and folder

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Split classes into their own files and folder
..


Split classes into their own files and folder

Classes are now their own file, as well as in the folder `/includes`.

Change-Id: I85bc4564081de310b443897bf97f6db017f834dd
---
A includes/GamepressSkinNavigationService.php
R includes/GamepressTemplate.php
A includes/SkinGamepress.php
M skin.json
4 files changed, 215 insertions(+), 213 deletions(-)

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



diff --git a/includes/GamepressSkinNavigationService.php 
b/includes/GamepressSkinNavigationService.php
new file mode 100644
index 000..132218a
--- /dev/null
+++ b/includes/GamepressSkinNavigationService.php
@@ -0,0 +1,168 @@
+forContent = $forContent;
+
+   $useCache = $wgLang->getCode() == $wgContLang->getCode();
+
+   if ( $useCache || $this->forContent ) {
+   $cacheKey = wfMemcKey( $messageName, self::version );
+   $nodes = $wgMemc->get( $cacheKey );
+   }
+
+   if ( empty( $nodes ) ) {
+   if ( $this->forContent ) {
+   $lines = explode( "\n", wfMessage( $messageName 
)->inContentLanguage()->text() );
+   } else {
+   $lines = explode( "\n", wfMessage( $messageName 
)->text() );
+   }
+   $nodes = $this->parseLines( $lines, $maxChildrenAtLevel 
);
+
+   if ( $useCache || $this->forContent ) {
+   $wgMemc->set( $cacheKey, $nodes, $duration );
+   }
+   }
+
+   return $nodes;
+   }
+
+   /**
+* Function used by parseMessage() above.
+*
+* @param $lines String: newline-separated lines from the supplied MW: 
msg
+* @param $maxChildrenAtLevel Array:
+* @return Array
+*/
+   private function parseLines( $lines, $maxChildrenAtLevel = array() ) {
+   $nodes = array();
+
+   if ( is_array( $lines ) && count( $lines ) > 0 ) {
+   $lastDepth = 0;
+   $i = 0;
+   $lastSkip = null;
+
+   foreach ( $lines as $line ) {
+   // we are interested only in lines that are not 
empty and start with asterisk
+   if ( trim( $line ) != '' && $line{0} == '*' ) {
+   $depth = strrpos( $line, '*' ) + 1;
+
+   if ( $lastSkip !== null && $depth >= 
$lastSkip ) {
+   continue;
+   } else {
+   $lastSkip = null;
+   }
+
+   if ( $depth == $lastDepth + 1 ) {
+   $parentIndex = $i;
+   } elseif ( $depth == $lastDepth ) {
+   $parentIndex = 
$nodes[$i]['parentIndex'];
+   } else {
+   for ( $x = $i; $x >= 0; $x-- ) {
+   if ( $x == 0 ) {
+   $parentIndex = 
0;
+   break;
+   }
+   if ( 
$nodes[$x]['depth'] <= $depth - 1 ) {
+   $parentIndex = 
$x;
+   break;
+   }
+   }
+   }
+
+   if ( isset( $maxChildrenAtLevel[$depth 
- 1] ) ) {
+   if ( isset( 
$nodes[$parentIndex]['children'] ) ) {
+   if ( count( 
$nodes[$parentIndex]['children'] ) >= $maxChildrenAtLevel[$depth - 1] ) {
+   $lastSkip = 
$depth;
+   continue;
+   }
+   }
+   }
+
+   $node = $this->parseOneLine( $line );
+   $node['parentIndex'] = $parentIndex;
+

[MediaWiki-commits] [Gerrit] mediawiki...Nimbus[master]: Split classes into their own files and folder

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Split classes into their own files and folder
..


Split classes into their own files and folder

Classes are now their own file, as well as in the folder `/includes`.

Change-Id: I83715539c7e7f1f76967a10813265286f3e6bc7a
---
R includes/NimbusTemplate.php
A includes/SkinNimbus.php
M skin.json
3 files changed, 36 insertions(+), 35 deletions(-)

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



diff --git a/Nimbus.skin.php b/includes/NimbusTemplate.php
similarity index 97%
rename from Nimbus.skin.php
rename to includes/NimbusTemplate.php
index d05e606..e0d0179 100644
--- a/Nimbus.skin.php
+++ b/includes/NimbusTemplate.php
@@ -21,38 +21,6 @@
 }
 
 /**
- * Inherit main code from SkinTemplate, set the CSS and template filter.
- * @ingroup Skins
- */
-class SkinNimbus extends SkinTemplate {
-   public $skinname = 'nimbus', $stylename = 'nimbus',
-   $template = 'NimbusTemplate', $useHeadElement = true;
-
-   /**
-* Load the JavaScript required by the menu and whatnot.
-*
-* @param OutputPage $out
-*/
-   public function initPage( OutputPage $out ) {
-   parent::initPage( $out );
-
-   $out->addModules( 'skins.nimbus.menu' );
-   }
-
-   function setupSkinUserCss( OutputPage $out ) {
-   parent::setupSkinUserCss( $out );
-
-   // Add CSS & JS
-   $out->addModuleStyles( array(
-   'mediawiki.skinning.interface',
-   'mediawiki.skinning.content.externallinks',
-   'skins.monobook.styles',
-   'skins.nimbus'
-   ) );
-   }
-}
-
-/**
  * Main skin class.
  * @ingroup Skins
  */
diff --git a/includes/SkinNimbus.php b/includes/SkinNimbus.php
new file mode 100644
index 000..cad0580
--- /dev/null
+++ b/includes/SkinNimbus.php
@@ -0,0 +1,33 @@
+addModules( 'skins.nimbus.menu' );
+   }
+
+   function setupSkinUserCss( OutputPage $out ) {
+   parent::setupSkinUserCss( $out );
+
+   // Add CSS & JS
+   $out->addModuleStyles( array(
+   'mediawiki.skinning.interface',
+   'mediawiki.skinning.content.externallinks',
+   'skins.monobook.styles',
+   'skins.nimbus'
+   ) );
+   }
+}
\ No newline at end of file
diff --git a/skin.json b/skin.json
index 18ae1b2..5addbb8 100644
--- a/skin.json
+++ b/skin.json
@@ -1,6 +1,6 @@
 {
"name": "Nimbus",
-   "version": "3.5.0",
+   "version": "3.5.1",
"author": [
"Aaron Wright",
"David Pean",
@@ -20,8 +20,8 @@
]
},
"AutoloadClasses": {
-   "NimbusTemplate": "Nimbus.skin.php",
-   "SkinNimbus": "Nimbus.skin.php"
+   "NimbusTemplate": "includes/NimbusTemplate.php",
+   "SkinNimbus": "includes/SkinNimbus.php"
},
"ResourceFileModulePaths": {
"localBasePath": "",

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I83715539c7e7f1f76967a10813265286f3e6bc7a
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/skins/Nimbus
Gerrit-Branch: master
Gerrit-Owner: SamanthaNguyen 
Gerrit-Reviewer: Jack Phoenix 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...DuskToDawn[master]: Split classes into their own files and folder

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Split classes into their own files and folder
..


Split classes into their own files and folder

Classes are now their own file, as well as in the folder `/includes`.

Change-Id: Ie5358423ff56b2458aed82886b4c18f8b7d3d5bb
---
R includes/DuskToDawnTemplate.php
A includes/SkinDuskToDawn.php
M skin.json
3 files changed, 32 insertions(+), 31 deletions(-)

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



diff --git a/DuskToDawn.skin.php b/includes/DuskToDawnTemplate.php
similarity index 93%
rename from DuskToDawn.skin.php
rename to includes/DuskToDawnTemplate.php
index 6cd58c3..40a127d 100644
--- a/DuskToDawn.skin.php
+++ b/includes/DuskToDawnTemplate.php
@@ -9,35 +9,6 @@
  * @see http://wp-themes.com/dusk-to-dawn/
  */
 
-/**
- * Inherit main code from SkinTemplate, set the CSS and template filter.
- *
- * @ingroup Skins
- */
-class SkinDuskToDawn extends SkinTemplate {
-   public $skinname = 'dusktodawn', $stylename = 'dusktodawn',
-   $template = 'DuskToDawnTemplate', $useHeadElement = true;
-
-   /**
-* @param $out OutputPage
-*/
-   function setupSkinUserCss( OutputPage $out ) {
-   global $wgStylePath;
-
-   parent::setupSkinUserCss( $out );
-
-   // Load CSS via ResourceLoader
-   $out->addModuleStyles( array(
-   'mediawiki.skinning.interface',
-   'mediawiki.skinning.content.externallinks',
-   'skins.dusktodawn'
-   ) );
-
-   // And JS too!
-   $out->addModuleScripts( 'skins.dusktodawn' );
-   }
-}
-
 class DuskToDawnTemplate extends BaseTemplate {
 
/**
diff --git a/includes/SkinDuskToDawn.php b/includes/SkinDuskToDawn.php
new file mode 100644
index 000..6c1f2e0
--- /dev/null
+++ b/includes/SkinDuskToDawn.php
@@ -0,0 +1,30 @@
+addModuleStyles( array(
+   'mediawiki.skinning.interface',
+   'mediawiki.skinning.content.externallinks',
+   'skins.dusktodawn'
+   ) );
+
+   // And JS too!
+   $out->addModuleScripts( 'skins.dusktodawn' );
+   }
+}
\ No newline at end of file
diff --git a/skin.json b/skin.json
index 6bf6389..bec5b2e 100644
--- a/skin.json
+++ b/skin.json
@@ -19,8 +19,8 @@
]
},
"AutoloadClasses": {
-   "SkinDuskToDawn": "DuskToDawn.skin.php",
-   "DuskToDawnTemplate": "DuskToDawn.skin.php"
+   "SkinDuskToDawn": "includes/SkinDuskToDawn.php",
+   "DuskToDawnTemplate": "includes/DuskToDawnTemplate.php"
},
"ResourceFileModulePaths": {
"localBasePath": "",

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie5358423ff56b2458aed82886b4c18f8b7d3d5bb
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/skins/DuskToDawn
Gerrit-Branch: master
Gerrit-Owner: SamanthaNguyen 
Gerrit-Reviewer: Jack Phoenix 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...Bouquet[master]: Split classes into their own files and folder

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Split classes into their own files and folder
..


Split classes into their own files and folder

Classes are now their own file, as well as in the folder `/includes`.

Change-Id: I980159f34bcd3d7f7fe0f5d2055ed3e06e6e4332
---
A includes/BouquetSkinNavigationService.php
R includes/BouquetTemplate.php
A includes/SkinBouquet.php
M skin.json
4 files changed, 209 insertions(+), 204 deletions(-)

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



diff --git a/includes/BouquetSkinNavigationService.php 
b/includes/BouquetSkinNavigationService.php
new file mode 100644
index 000..e677f96
--- /dev/null
+++ b/includes/BouquetSkinNavigationService.php
@@ -0,0 +1,168 @@
+forContent = $forContent;
+
+   $useCache = $wgLang->getCode() == $wgContLang->getCode();
+
+   if ( $useCache || $this->forContent ) {
+   $cacheKey = wfMemcKey( $messageName, self::version );
+   $nodes = $wgMemc->get( $cacheKey );
+   }
+
+   if ( empty( $nodes ) ) {
+   if ( $this->forContent ) {
+   $lines = explode( "\n", wfMessage( $messageName 
)->inContentLanguage()->text() );
+   } else {
+   $lines = explode( "\n", wfMessage( $messageName 
)->text() );
+   }
+   $nodes = $this->parseLines( $lines, $maxChildrenAtLevel 
);
+
+   if ( $useCache || $this->forContent ) {
+   $wgMemc->set( $cacheKey, $nodes, $duration );
+   }
+   }
+
+   return $nodes;
+   }
+
+   /**
+* Function used by parseMessage() above.
+*
+* @param $lines String: newline-separated lines from the supplied MW: 
msg
+* @param $maxChildrenAtLevel Array:
+* @return Array
+*/
+   private function parseLines( $lines, $maxChildrenAtLevel = array() ) {
+   $nodes = array();
+
+   if ( is_array( $lines ) && count( $lines ) > 0 ) {
+   $lastDepth = 0;
+   $i = 0;
+   $lastSkip = null;
+
+   foreach ( $lines as $line ) {
+   // we are interested only in lines that are not 
empty and start with asterisk
+   if ( trim( $line ) != '' && $line{0} == '*' ) {
+   $depth = strrpos( $line, '*' ) + 1;
+
+   if ( $lastSkip !== null && $depth >= 
$lastSkip ) {
+   continue;
+   } else {
+   $lastSkip = null;
+   }
+
+   if ( $depth == $lastDepth + 1 ) {
+   $parentIndex = $i;
+   } elseif ( $depth == $lastDepth ) {
+   $parentIndex = 
$nodes[$i]['parentIndex'];
+   } else {
+   for ( $x = $i; $x >= 0; $x-- ) {
+   if ( $x == 0 ) {
+   $parentIndex = 
0;
+   break;
+   }
+   if ( 
$nodes[$x]['depth'] <= $depth - 1 ) {
+   $parentIndex = 
$x;
+   break;
+   }
+   }
+   }
+
+   if ( isset( $maxChildrenAtLevel[$depth 
- 1] ) ) {
+   if ( isset( 
$nodes[$parentIndex]['children'] ) ) {
+   if ( count( 
$nodes[$parentIndex]['children'] ) >= $maxChildrenAtLevel[$depth - 1] ) {
+   $lastSkip = 
$depth;
+   continue;
+   }
+   }
+   }
+
+   $node = $this->parseOneLine( $line );
+   $node['parentIndex'] = $parentIndex;
+

[MediaWiki-commits] [Gerrit] mediawiki...Dusk[master]: Split classes into their own files and folder

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Split classes into their own files and folder
..


Split classes into their own files and folder

Classes are now their own file, as well as in the folder `/includes`.

Change-Id: I0e7b0400d24ce8f22656982707fabee549e58962
---
R includes/DuskTemplate.php
A includes/SkinDusk.php
M skin.json
3 files changed, 25 insertions(+), 29 deletions(-)

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



diff --git a/Dusk.skin.php b/includes/DuskTemplate.php
similarity index 92%
rename from Dusk.skin.php
rename to includes/DuskTemplate.php
index 6370c08..430521f 100644
--- a/Dusk.skin.php
+++ b/includes/DuskTemplate.php
@@ -7,32 +7,6 @@
  * @ingroup Skins
  */
 
-/**
- * Inherit main code from SkinTemplate, set the CSS and template filter.
- *
- * @ingroup Skins
- */
-class SkinDusk extends SkinTemplate {
-   public $skinname = 'dusk', $stylename = 'dusk',
-   $template = 'DuskTemplate', $useHeadElement = true;
-
-   function setupSkinUserCss( OutputPage $out ) {
-   parent::setupSkinUserCss( $out );
-
-   // Add CSS via ResourceLoader
-   $out->addModuleStyles( array(
-   'mediawiki.skinning.interface',
-   'mediawiki.skinning.content.externallinks',
-   'skins.dusk'
-   ) );
-   }
-}
-
-/**
- * Main Dusk skin class.
- *
- * @ingroup Skins
- */
 class DuskTemplate extends BaseTemplate {
/**
 * Template filter callback for the Dusk skin.
diff --git a/includes/SkinDusk.php b/includes/SkinDusk.php
new file mode 100644
index 000..ebed351
--- /dev/null
+++ b/includes/SkinDusk.php
@@ -0,0 +1,22 @@
+addModuleStyles( array(
+   'mediawiki.skinning.interface',
+   'mediawiki.skinning.content.externallinks',
+   'skins.dusk'
+   ) );
+   }
+}
\ No newline at end of file
diff --git a/skin.json b/skin.json
index 1493966..1f75bf7 100644
--- a/skin.json
+++ b/skin.json
@@ -1,6 +1,6 @@
 {
"name": "Dusk",
-   "version": "3.3.0",
+   "version": "3.3.1",
"author": [
"Gregory S. Hayes",
"Becca Wei",
@@ -23,8 +23,8 @@
"dusk": [ "green" ]
},
"AutoloadClasses": {
-   "SkinDusk": "Dusk.skin.php",
-   "DuskTemplate": "Dusk.skin.php"
+   "SkinDusk": "includes/SkinDusk.php",
+   "DuskTemplate": "includes/DuskTemplate.php"
},
"ResourceFileModulePaths": {
"localBasePath": "",

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0e7b0400d24ce8f22656982707fabee549e58962
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/skins/Dusk
Gerrit-Branch: master
Gerrit-Owner: SamanthaNguyen 
Gerrit-Reviewer: Jack Phoenix 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...DeskMessMirrored[master]: Split classes into their own files and folder

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Split classes into their own files and folder
..


Split classes into their own files and folder

Classes are now their own file, as well as in the folder `/includes`.

Change-Id: I6d0b3cbb1fee95b78b9df0bb05baf2e7670dc1d5
---
A includes/DeskMessMirroredSkinNavigationService.php
R includes/DeskMessMirroredTemplate.php
A includes/SkinDeskMessMirrored.php
M skin.json
4 files changed, 215 insertions(+), 213 deletions(-)

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



diff --git a/includes/DeskMessMirroredSkinNavigationService.php 
b/includes/DeskMessMirroredSkinNavigationService.php
new file mode 100644
index 000..73e86b3
--- /dev/null
+++ b/includes/DeskMessMirroredSkinNavigationService.php
@@ -0,0 +1,168 @@
+forContent = $forContent;
+
+   $useCache = $wgLang->getCode() == $wgContLang->getCode();
+
+   if ( $useCache || $this->forContent ) {
+   $cacheKey = wfMemcKey( $messageName, self::version );
+   $nodes = $wgMemc->get( $cacheKey );
+   }
+
+   if ( empty( $nodes ) ) {
+   if ( $this->forContent ) {
+   $lines = explode( "\n", wfMessage( $messageName 
)->inContentLanguage()->text() );
+   } else {
+   $lines = explode( "\n", wfMessage( $messageName 
)->text() );
+   }
+   $nodes = $this->parseLines( $lines, $maxChildrenAtLevel 
);
+
+   if ( $useCache || $this->forContent ) {
+   $wgMemc->set( $cacheKey, $nodes, $duration );
+   }
+   }
+
+   return $nodes;
+   }
+
+   /**
+* Function used by parseMessage() above.
+*
+* @param $lines String: newline-separated lines from the supplied MW: 
msg
+* @param $maxChildrenAtLevel Array:
+* @return Array
+*/
+   private function parseLines( $lines, $maxChildrenAtLevel = array() ) {
+   $nodes = array();
+
+   if ( is_array( $lines ) && count( $lines ) > 0 ) {
+   $lastDepth = 0;
+   $i = 0;
+   $lastSkip = null;
+
+   foreach ( $lines as $line ) {
+   // we are interested only in lines that are not 
empty and start with asterisk
+   if ( trim( $line ) != '' && $line{0} == '*' ) {
+   $depth = strrpos( $line, '*' ) + 1;
+
+   if ( $lastSkip !== null && $depth >= 
$lastSkip ) {
+   continue;
+   } else {
+   $lastSkip = null;
+   }
+
+   if ( $depth == $lastDepth + 1 ) {
+   $parentIndex = $i;
+   } elseif ( $depth == $lastDepth ) {
+   $parentIndex = 
$nodes[$i]['parentIndex'];
+   } else {
+   for ( $x = $i; $x >= 0; $x-- ) {
+   if ( $x == 0 ) {
+   $parentIndex = 
0;
+   break;
+   }
+   if ( 
$nodes[$x]['depth'] <= $depth - 1 ) {
+   $parentIndex = 
$x;
+   break;
+   }
+   }
+   }
+
+   if ( isset( $maxChildrenAtLevel[$depth 
- 1] ) ) {
+   if ( isset( 
$nodes[$parentIndex]['children'] ) ) {
+   if ( count( 
$nodes[$parentIndex]['children'] ) >= $maxChildrenAtLevel[$depth - 1] ) {
+   $lastSkip = 
$depth;
+   continue;
+   }
+   }
+   }
+
+   $node = $this->parseOneLine( $line );
+   $no

[MediaWiki-commits] [Gerrit] integration/config[master]: Archive repo mediawiki/extensions/ThumbParser

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Archive repo mediawiki/extensions/ThumbParser
..


Archive repo mediawiki/extensions/ThumbParser

Bug: T65377
Change-Id: Ic9b9f46e5664fa1d656b0876ec35dbbce135ea24
---
M zuul/layout.yaml
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/zuul/layout.yaml b/zuul/layout.yaml
index 290645d..d569a0e 100644
--- a/zuul/layout.yaml
+++ b/zuul/layout.yaml
@@ -5752,7 +5752,7 @@
 
   - name: mediawiki/extensions/ThumbParser
 template:
-  - name: extension-unittests-generic
+  - name: archived
 
   - name: mediawiki/extensions/TimelineTable
 template:

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic9b9f46e5664fa1d656b0876ec35dbbce135ea24
Gerrit-PatchSet: 3
Gerrit-Project: integration/config
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Hashar 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...VisualEditor[master]: Update VE core submodule to master (6741337)

2017-11-04 Thread Esanders (Code Review)

Esanders has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389233 )

Change subject: Update VE core submodule to master (6741337)
..

Update VE core submodule to master (6741337)

New changes:
0261c91 Localisation updates from https://translatewiki.net.
5043c49 ve.ui.Surface: Fix documentation for #getDom
d6fece6 Localisation updates from https://translatewiki.net.
5b46356 ve.BranchNode#setDocument: Fix highly misleading comment
0cb8c98 Update OOjs UI to v0.24.1
2b401cf Localisation updates from https://translatewiki.net.
e36f398 Add method for converting plaintext offsets to source-mode offsets
6741337 Fix typo in unit test

Change-Id: I1071faeec586fa2b44be810a35f8044800c8f800
---
M lib/ve
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor 
refs/changes/33/389233/1

diff --git a/lib/ve b/lib/ve
index 8d1c0d0..6741337 16
--- a/lib/ve
+++ b/lib/ve
@@ -1 +1 @@
-Subproject commit 8d1c0d0260a313292016665b947b9bc5f365fb6d
+Subproject commit 6741337ffaa180c3099be45d8abccfb3b28e94f8

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1071faeec586fa2b44be810a35f8044800c8f800
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Esanders 

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

[MediaWiki-commits] [Gerrit] VisualEditor/VisualEditor[master]: Convert empty doc to empty string in convertToSource

2017-11-04 Thread Esanders (Code Review)

Esanders has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389232 )

Change subject: Convert empty doc to empty string in convertToSource
..

Convert empty doc to empty string in convertToSource

Change-Id: Ie034a30572f2dafcb8882b4e0af4df617040f2ed
---
M src/dm/ve.dm.SourceSurfaceFragment.js
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/VisualEditor/VisualEditor 
refs/changes/32/389232/1

diff --git a/src/dm/ve.dm.SourceSurfaceFragment.js 
b/src/dm/ve.dm.SourceSurfaceFragment.js
index 243eba0..bcb9ddf 100644
--- a/src/dm/ve.dm.SourceSurfaceFragment.js
+++ b/src/dm/ve.dm.SourceSurfaceFragment.js
@@ -225,7 +225,7 @@
  */
 ve.dm.SourceSurfaceFragment.prototype.convertToSource = function ( doc ) {
if ( !doc.data.hasContent() ) {
-   return $.Deferred().reject().promise();
+   return $.Deferred().resolve( '' ).promise();
} else {
return $.Deferred().resolve(
ve.properInnerHtml(

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie034a30572f2dafcb8882b4e0af4df617040f2ed
Gerrit-PatchSet: 1
Gerrit-Project: VisualEditor/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Esanders 

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

[MediaWiki-commits] [Gerrit] mediawiki...VisualEditor[master]: Convert empty doc to empty string in convertToSource

2017-11-04 Thread Esanders (Code Review)

Esanders has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389231 )

Change subject: Convert empty doc to empty string in convertToSource
..

Convert empty doc to empty string in convertToSource

Change-Id: I8753169ff206b5ce23cfd4c0dc7f97ac6fd1fff0
---
M modules/ve-mw/dm/ve.dm.MWWikitextSurfaceFragment.js
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor 
refs/changes/31/389231/1

diff --git a/modules/ve-mw/dm/ve.dm.MWWikitextSurfaceFragment.js 
b/modules/ve-mw/dm/ve.dm.MWWikitextSurfaceFragment.js
index bfe2139..f102637 100644
--- a/modules/ve-mw/dm/ve.dm.MWWikitextSurfaceFragment.js
+++ b/modules/ve-mw/dm/ve.dm.MWWikitextSurfaceFragment.js
@@ -131,7 +131,7 @@
var wikitextPromise;
 
if ( !doc.data.hasContent() ) {
-   return $.Deferred().reject().promise();
+   return $.Deferred().resolve( '' ).promise();
}
 
wikitextPromise = ve.init.target.getWikitextFragment( doc, false );

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8753169ff206b5ce23cfd4c0dc7f97ac6fd1fff0
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Esanders 

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

[MediaWiki-commits] [Gerrit] mediawiki...SpellingDictionary[master]: build: Adding MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/388854 )

Change subject: build: Adding MinusX
..


build: Adding MinusX

Change-Id: I203792744940d7ef2f49ccac56682e441ac58ecb
---
M composer.json
1 file changed, 7 insertions(+), 2 deletions(-)

Approvals:
  Umherirrender: Verified
  Legoktm: Looks good to me, approved



diff --git a/composer.json b/composer.json
index 686b65b..6515194 100644
--- a/composer.json
+++ b/composer.json
@@ -1,11 +1,16 @@
 {
"require-dev": {
"jakub-onderka/php-parallel-lint": "0.9.2",
-   "jakub-onderka/php-console-highlighter": "0.3.2"
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
},
"scripts": {
"test": [
-   "parallel-lint . --exclude vendor"
+   "parallel-lint . --exclude vendor",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
]
}
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I203792744940d7ef2f49ccac56682e441ac58ecb
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/SpellingDictionary
Gerrit-Branch: master
Gerrit-Owner: Legoktm 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...Echo[master]: Unify SVG markup

2017-11-04 Thread VolkerE (Code Review)

VolkerE has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389230 )

Change subject: Unify SVG markup
..

Unify SVG markup

Align SVG markup across Foundation products by:
- unifying XML declaration,
- removing unnecessary `standalone="no"` attribute as it's default,
- adding `width` and `height` attributes where missing,
- removing all `g` group elements where applicable,
- bringing attributes in order,
- removing unnecessary ids and
- unifying whitespace.

Bug: T178867
Change-Id: Id3799a46c14f793636ca9bca3ad0f642466ed5a0
---
M images/betafeatures-icon-notifications-ltr.svg
M images/betafeatures-icon-notifications-rtl.svg
M modules/icons/bell.svg
M modules/icons/bellOn-ltr.svg
M modules/icons/bellOn-rtl.svg
M modules/icons/changes.svg
M modules/icons/chat.svg
M modules/icons/double-check.svg
M modules/icons/edit-user-talk.svg
M modules/icons/emailuser.svg
M modules/icons/feedback.svg
M modules/icons/generic.svg
M modules/icons/global.svg
M modules/icons/help.svg
M modules/icons/link-blue.svg
M modules/icons/mention-failure.svg
M modules/icons/mention-status-bundle.svg
M modules/icons/mention-success.svg
M modules/icons/mention.svg
M modules/icons/ooui-edit-ltr-progressive.svg
M modules/icons/ooui-edit-rtl-progressive.svg
M modules/icons/revert.svg
M modules/icons/reviewed.svg
M modules/icons/speechBubbles-ltr.svg
M modules/icons/speechBubbles-rtl.svg
M modules/icons/thanks-ltr.svg
M modules/icons/thanks-rtl.svg
M modules/icons/trash.svg
M modules/icons/tray.svg
M modules/icons/user-rights.svg
M modules/icons/user-speech-bubble.svg
31 files changed, 134 insertions(+), 140 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Echo 
refs/changes/30/389230/1

diff --git a/images/betafeatures-icon-notifications-ltr.svg 
b/images/betafeatures-icon-notifications-ltr.svg
index 2a55cd9..52a12b2 100644
--- a/images/betafeatures-icon-notifications-ltr.svg
+++ b/images/betafeatures-icon-notifications-ltr.svg
@@ -1,27 +1,27 @@
-
-http://www.w3.org/2000/svg"; width="264" height="162" viewBox="0 0 
264 162" id="svg2">
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+http://www.w3.org/2000/svg"; width="264" height="162" viewBox="0 0 
264 162">
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
 
diff --git a/images/betafeatures-icon-notifications-rtl.svg 
b/images/betafeatures-icon-notifications-rtl.svg
index e6d2be7..52683b6 100644
--- a/images/betafeatures-icon-notifications-rtl.svg
+++ b/images/betafeatures-icon-notifications-rtl.svg
@@ -1,27 +1,25 @@
-
-http://www.w3.org/2000/svg"; width="264" height="162" viewBox="0 0 
264 162" id="svg2">
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+http://www.w3.org/2000/svg"; width="264" height="162" viewBox="0 0 
264 162">
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
+   
 
diff --git a/modules/icons/bell.svg b/modules/icons/bell.svg
index 787ed14..efb0e82 100644
--- a/modules/icons/bell.svg
+++ b/modules/icons/bell.svg
@@ -1,4 +1,4 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
+   
 
diff --git a/modules/icons/bellOn-ltr.svg b/modules/icons/bellOn-ltr.svg
index f68a8e0..d4e3bcf 100644
--- a/modules/icons/bellOn-ltr.svg
+++ b/modules/icons/bellOn-ltr.svg
@@ -1,4 +1,4 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
+   
 
diff --git a/modules/icons/bellOn-rtl.svg b/modules/icons/bellOn-rtl.svg
index 92bcef5..44a35c6 100644
--- a/modules/icons/bellOn-rtl.svg
+++ b/modules/icons/bellOn-rtl.svg
@@ -1,4 +1,4 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" viewBox="0 0 24 
24">
-
+   
 
diff --git a/modules/icons/changes.svg b/modules/icons/changes.svg
index efe6528..0642f5d 100644
--- a/modules/icons/changes.svg
+++ b/modules/icons/changes.svg
@@ -1,4 +1,4 @@
-
+
 http://www.w3.org/2000/svg"; width="24" height="24" view

[MediaWiki-commits] [Gerrit] mediawiki...YetAnotherKeywords[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I2a0b1e8915c6de7a57bed42009345ec4b60fe5b1
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 1 deletion(-)

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



diff --git a/.gitignore b/.gitignore
index a7abe44..8ec4b92 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,3 @@
 node_modules/
 vendor/
-
+composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2a0b1e8915c6de7a57bed42009345ec4b60fe5b1
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/YetAnotherKeywords
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Fix function annotiation

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Fix function annotiation
..


Fix function annotiation

Change-Id: I4dfc62c8ef2226b46aa1eed508816377f6599ab2
---
M includes/changes/ChangesListStringOptionsFilterGroup.php
1 file changed, 2 insertions(+), 1 deletion(-)

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



diff --git a/includes/changes/ChangesListStringOptionsFilterGroup.php 
b/includes/changes/ChangesListStringOptionsFilterGroup.php
index 653344a..775fd76 100644
--- a/includes/changes/ChangesListStringOptionsFilterGroup.php
+++ b/includes/changes/ChangesListStringOptionsFilterGroup.php
@@ -239,7 +239,8 @@
/**
 * Check if this filter group is currently active
 *
-* @param {boolean} $isStructuredUI Is structured filters UI current 
enabled
+* @param bool $isStructuredUI Is structured filters UI current enabled
+* @return bool
 */
private function isActive( $isStructuredUI ) {
// STRING_OPTIONS filter groups are exclusively active on 
Structured UI

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4dfc62c8ef2226b46aa1eed508816377f6599ab2
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: MaxSem 
Gerrit-Reviewer: Krinkle 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...YotpoReviews[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I7e712b63fb8df9a324bc7ee3959a2301ce49d749
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 1 deletion(-)

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



diff --git a/.gitignore b/.gitignore
index a7abe44..8ec4b92 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,3 @@
 node_modules/
 vendor/
-
+composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7e712b63fb8df9a324bc7ee3959a2301ce49d749
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/YotpoReviews
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki/core[master]: Remove use of a nonexistent variable

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: Remove use of a nonexistent variable
..


Remove use of a nonexistent variable

Was gone since I4c22666fb98 year and a half ago.

Change-Id: I7e2fb015310ea2541cd4a92133627b748507ef78
---
M includes/specials/SpecialJavaScriptTest.php
1 file changed, 0 insertions(+), 9 deletions(-)

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



diff --git a/includes/specials/SpecialJavaScriptTest.php 
b/includes/specials/SpecialJavaScriptTest.php
index 17c64c8..b786c86 100644
--- a/includes/specials/SpecialJavaScriptTest.php
+++ b/includes/specials/SpecialJavaScriptTest.php
@@ -199,15 +199,6 @@
echo $html;
}
 
-   /**
-* Return an array of subpages that this special page will accept.
-*
-* @return string[] subpages
-*/
-   public function getSubpagesForPrefixSearch() {
-   return self::$frameworks;
-   }
-
protected function getGroupName() {
return 'other';
}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7e2fb015310ea2541cd4a92133627b748507ef78
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: MaxSem 
Gerrit-Reviewer: Florianschmidtwelzow 
Gerrit-Reviewer: Krinkle 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...SemanticSifter[master]: build: Adding MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/388827 )

Change subject: build: Adding MinusX
..


build: Adding MinusX

Change-Id: I619796ac7bcc97f786a1c8b17edfcc6b393b4b3f
---
M composer.json
1 file changed, 9 insertions(+), 4 deletions(-)

Approvals:
  Umherirrender: Verified
  Legoktm: Looks good to me, approved



diff --git a/composer.json b/composer.json
index c1229ad..9911cc2 100644
--- a/composer.json
+++ b/composer.json
@@ -17,20 +17,25 @@
}
],
"autoload": {
-   "files" : [
+   "files": [
"SemanticSifter.php"
],
-   "classmap":[
+   "classmap": [
"src/"
]
},
"require-dev": {
"jakub-onderka/php-parallel-lint": "0.9.2",
-   "jakub-onderka/php-console-highlighter": "0.3.2"
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
},
"scripts": {
"test": [
-   "parallel-lint . --exclude vendor --exclude 
node_modules --exclude extensions"
+   "parallel-lint . --exclude vendor --exclude 
node_modules --exclude extensions",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
]
}
 }

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I619796ac7bcc97f786a1c8b17edfcc6b393b4b3f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/SemanticSifter
Gerrit-Branch: master
Gerrit-Owner: Legoktm 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WhiteSpace[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I4c97ada3b8b1dbc707c9567f0a9b5d397b2f11d7
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4c97ada3b8b1dbc707c9567f0a9b5d397b2f11d7
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WhiteSpace
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WikivoteMapsYandex[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I410017187d15ed8893ce0fcbad29f2806968c0ab
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I410017187d15ed8893ce0fcbad29f2806968c0ab
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikivoteMapsYandex
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WikiObjectModel[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I650558b38119fea7b71517c39b0be4db1a08eb41
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I650558b38119fea7b71517c39b0be4db1a08eb41
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikiObjectModel
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WikiCategoryTagCloud[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I4d7ec8da80b4238c6f3764c3150f3e8eef2b8311
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4d7ec8da80b4238c6f3764c3150f3e8eef2b8311
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikiCategoryTagCloud
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WikiTextLoggedInOut[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I97d6f151c5c965954778e310ecd4885229d0e94e
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I97d6f151c5c965954778e310ecd4885229d0e94e
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikiTextLoggedInOut
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WhoIsWatching[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I9a74d925024ec0662d4315a5be25e72ebd194453
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index 2f5650e..ec11425 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9a74d925024ec0662d4315a5be25e72ebd194453
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/WhoIsWatching
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WikEdDiff[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I03024403067433fb3300badcb0aa20117d96bdfd
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 1 deletion(-)

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



diff --git a/.gitignore b/.gitignore
index a7abe44..8ec4b92 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,3 @@
 node_modules/
 vendor/
-
+composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I03024403067433fb3300badcb0aa20117d96bdfd
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikEdDiff
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WhitelistPages[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I5203c068930d265ca02d250611788ae4812bbdf3
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5203c068930d265ca02d250611788ae4812bbdf3
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WhitelistPages
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WhosOnline[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: Ibc8841d11008c0f6fc6fb6484a44f8f07a7e3d96
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 1 deletion(-)

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



diff --git a/.gitignore b/.gitignore
index a7abe44..8ec4b92 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,3 @@
 node_modules/
 vendor/
-
+composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibc8841d11008c0f6fc6fb6484a44f8f07a7e3d96
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WhosOnline
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WikiArticleFeeds[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: Ide0a7e5bc510169195a1b6a0c01024f7d7e6b1a6
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ide0a7e5bc510169195a1b6a0c01024f7d7e6b1a6
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikiArticleFeeds
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WikiLovesMonuments[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I8469acec653d2eb97c463edf6a453298eec699d6
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8469acec653d2eb97c463edf6a453298eec699d6
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikiLovesMonuments
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WikiForum[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I2d6fca9018e14a109e840d27a97810bb28f11670
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2d6fca9018e14a109e840d27a97810bb28f11670
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikiForum
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: UltrasonicNXT 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WantedPagesFromNS[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I5c5ff4411e9bc1c3412f10f1abe34f7f6cbf2654
---
M .gitignore
A composer.json
2 files changed, 18 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index 07e6e47..db8a779 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1 +1,3 @@
 /node_modules
+/vendor
+/composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5c5ff4411e9bc1c3412f10f1abe34f7f6cbf2654
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WantedPagesFromNS
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WatchSubpages[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: Iaa5d190e457fbea469d7dced8a40d3e7c5946837
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 1 deletion(-)

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



diff --git a/.gitignore b/.gitignore
index a7abe44..8ec4b92 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,3 @@
 node_modules/
 vendor/
-
+composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iaa5d190e457fbea469d7dced8a40d3e7c5946837
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WatchSubpages
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...VoteNY[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: Ic9716a3a1ecbb5fcff39940148abeca7e58969b9
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic9716a3a1ecbb5fcff39940148abeca7e58969b9
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VoteNY
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...VirtualKeyboard[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I2658d2852d2a98640aa5036430b997885d9f5053
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 1 deletion(-)

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



diff --git a/.gitignore b/.gitignore
index a7abe44..8ec4b92 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,3 @@
 node_modules/
 vendor/
-
+composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2658d2852d2a98640aa5036430b997885d9f5053
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VirtualKeyboard
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...ViewFiles[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I8a0bc2ea9d4b92258aaf65f77565a1586d63d69d
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8a0bc2ea9d4b92258aaf65f77565a1586d63d69d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ViewFiles
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...Video[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I91e1eb8b47f356be2945ba45dec8de32e809d193
---
A composer.json
1 file changed, 16 insertions(+), 0 deletions(-)

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



diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I91e1eb8b47f356be2945ba45dec8de32e809d193
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Video
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...UserPageViewTracker[master]: build: Add php-parallel-lint

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint
..


build: Add php-parallel-lint

php-lint will test for valid php files

Change-Id: I7c9418217e82299a8e70857b419c2a94d1c1698c
---
M .gitignore
A composer.json
2 files changed, 12 insertions(+), 1 deletion(-)

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



diff --git a/.gitignore b/.gitignore
index a7abe44..8ec4b92 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,3 @@
 node_modules/
 vendor/
-
+composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..aebd773
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,11 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules"
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7c9418217e82299a8e70857b419c2a94d1c1698c
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/UserPageViewTracker
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...UserThrottle[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: Iec3f06b3bda24e63d5ce0311d215f83481554ece
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iec3f06b3bda24e63d5ce0311d215f83481554ece
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/UserThrottle
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...UserStatus[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: Ia56a9ac10ac8d13850945f41bc489356bc53c0b7
---
M .gitignore
A composer.json
2 files changed, 18 insertions(+), 0 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index 2ccbe46..47e36d5 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1 +1,3 @@
 /node_modules/
+/vendor/
+/composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia56a9ac10ac8d13850945f41bc489356bc53c0b7
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/UserStatus
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...WikivoteMapsYandex[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389224 )

Change subject: build: Add php-parallel-lint and MinusX
..

build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I410017187d15ed8893ce0fcbad29f2806968c0ab
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WikivoteMapsYandex 
refs/changes/24/389224/1

diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I410017187d15ed8893ce0fcbad29f2806968c0ab
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikivoteMapsYandex
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...YetAnotherKeywords[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389225 )

Change subject: build: Add php-parallel-lint and MinusX
..

build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I2a0b1e8915c6de7a57bed42009345ec4b60fe5b1
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 1 deletion(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/YetAnotherKeywords 
refs/changes/25/389225/1

diff --git a/.gitignore b/.gitignore
index a7abe44..8ec4b92 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,3 @@
 node_modules/
 vendor/
-
+composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2a0b1e8915c6de7a57bed42009345ec4b60fe5b1
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/YetAnotherKeywords
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...UnusedRedirects[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread jenkins-bot (Code Review)

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

Change subject: build: Add php-parallel-lint and MinusX
..


build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: Ic3e0d9332bba6fb8d3608074e620fa9820786c91
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 1 deletion(-)

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



diff --git a/.gitignore b/.gitignore
index a7abe44..8ec4b92 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,3 @@
 node_modules/
 vendor/
-
+composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic3e0d9332bba6fb8d3608074e620fa9820786c91
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/UnusedRedirects
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
Gerrit-Reviewer: Umherirrender 
Gerrit-Reviewer: jenkins-bot <>

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

[MediaWiki-commits] [Gerrit] mediawiki...YotpoReviews[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389226 )

Change subject: build: Add php-parallel-lint and MinusX
..

build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I7e712b63fb8df9a324bc7ee3959a2301ce49d749
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/YotpoReviews 
refs/changes/26/389226/1

diff --git a/.gitignore b/.gitignore
index a7abe44..8ec4b92 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,3 @@
 node_modules/
 vendor/
-
+composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7e712b63fb8df9a324bc7ee3959a2301ce49d749
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/YotpoReviews
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...WikiTextLoggedInOut[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389223 )

Change subject: build: Add php-parallel-lint and MinusX
..

build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I97d6f151c5c965954778e310ecd4885229d0e94e
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WikiTextLoggedInOut 
refs/changes/23/389223/1

diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I97d6f151c5c965954778e310ecd4885229d0e94e
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikiTextLoggedInOut
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...WikiObjectModel[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389222 )

Change subject: build: Add php-parallel-lint and MinusX
..

build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I650558b38119fea7b71517c39b0be4db1a08eb41
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WikiObjectModel 
refs/changes/22/389222/1

diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I650558b38119fea7b71517c39b0be4db1a08eb41
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikiObjectModel
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...WikiForum[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389220 )

Change subject: build: Add php-parallel-lint and MinusX
..

build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I2d6fca9018e14a109e840d27a97810bb28f11670
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WikiForum 
refs/changes/20/389220/1

diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2d6fca9018e14a109e840d27a97810bb28f11670
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikiForum
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...WikiArticleFeeds[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389218 )

Change subject: build: Add php-parallel-lint and MinusX
..

build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: Ide0a7e5bc510169195a1b6a0c01024f7d7e6b1a6
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WikiArticleFeeds 
refs/changes/18/389218/1

diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ide0a7e5bc510169195a1b6a0c01024f7d7e6b1a6
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikiArticleFeeds
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...WikEdDiff[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389217 )

Change subject: build: Add php-parallel-lint and MinusX
..

build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I03024403067433fb3300badcb0aa20117d96bdfd
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WikEdDiff 
refs/changes/17/389217/1

diff --git a/.gitignore b/.gitignore
index a7abe44..8ec4b92 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,3 @@
 node_modules/
 vendor/
-
+composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I03024403067433fb3300badcb0aa20117d96bdfd
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikEdDiff
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...WikiCategoryTagCloud[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389219 )

Change subject: build: Add php-parallel-lint and MinusX
..

build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I4d7ec8da80b4238c6f3764c3150f3e8eef2b8311
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WikiCategoryTagCloud 
refs/changes/19/389219/1

diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4d7ec8da80b4238c6f3764c3150f3e8eef2b8311
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikiCategoryTagCloud
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...WikiLovesMonuments[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389221 )

Change subject: build: Add php-parallel-lint and MinusX
..

build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I8469acec653d2eb97c463edf6a453298eec699d6
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WikiLovesMonuments 
refs/changes/21/389221/1

diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8469acec653d2eb97c463edf6a453298eec699d6
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WikiLovesMonuments
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...WhosOnline[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389216 )

Change subject: build: Add php-parallel-lint and MinusX
..

build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: Ibc8841d11008c0f6fc6fb6484a44f8f07a7e3d96
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WhosOnline 
refs/changes/16/389216/1

diff --git a/.gitignore b/.gitignore
index a7abe44..8ec4b92 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,3 @@
 node_modules/
 vendor/
-
+composer.lock
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibc8841d11008c0f6fc6fb6484a44f8f07a7e3d96
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WhosOnline
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

[MediaWiki-commits] [Gerrit] mediawiki...WhiteSpace[master]: build: Add php-parallel-lint and MinusX

2017-11-04 Thread Umherirrender (Code Review)

Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/389214 )

Change subject: build: Add php-parallel-lint and MinusX
..

build: Add php-parallel-lint and MinusX

php-lint will test for valid php files
MinusX will test for executable bits

Change-Id: I4c97ada3b8b1dbc707c9567f0a9b5d397b2f11d7
---
M .gitignore
A composer.json
2 files changed, 17 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/WhiteSpace 
refs/changes/14/389214/1

diff --git a/.gitignore b/.gitignore
index e62fc28..9dfe80b 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,5 +1,6 @@
 node_modules/
 vendor/
+composer.lock
 
 .svn
 *~
diff --git a/composer.json b/composer.json
new file mode 100644
index 000..392434c
--- /dev/null
+++ b/composer.json
@@ -0,0 +1,16 @@
+{
+   "require-dev": {
+   "jakub-onderka/php-parallel-lint": "0.9.2",
+   "jakub-onderka/php-console-highlighter": "0.3.2",
+   "mediawiki/minus-x": "0.2.0"
+   },
+   "scripts": {
+   "test": [
+   "parallel-lint . --exclude vendor --exclude 
node_modules",
+   "minus-x check ."
+   ],
+   "fix": [
+   "minus-x fix ."
+   ]
+   }
+}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4c97ada3b8b1dbc707c9567f0a9b5d397b2f11d7
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/WhiteSpace
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 

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

1 2 3 4 5 6 7 >

1 - 100 of 647 matches

Mail list logo