[MediaWiki-commits] [Gerrit] mediawiki...CharRangeSpan[master]: Add php-parallel-lint

2017-09-30 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/376774 )

Change subject: Add php-parallel-lint
..


Add php-parallel-lint

php-lint will test for valid php files

Fix hook by using variable as reference

Change-Id: I7b1d634c9eb52d351d39f57522b2d2e8198f9b73
---
M .gitignore
M CharRangeSpan.body.php
M Gruntfile.js
A composer.json
4 files changed, 16 insertions(+), 2 deletions(-)

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



diff --git a/.gitignore b/.gitignore
index 1689d7a..8ea3b90 100644
--- a/.gitignore
+++ b/.gitignore
@@ -2,3 +2,5 @@
 *.kate-swp
 .*.swp
 node_modules/**
+vendor/**
+composer.lock
diff --git a/CharRangeSpan.body.php b/CharRangeSpan.body.php
index 7326791..1939cfa 100644
--- a/CharRangeSpan.body.php
+++ b/CharRangeSpan.body.php
@@ -11,7 +11,7 @@
 * Only public entry point of this class.
 * Called from ParserBeforeTidy
 */
-   public static function doCharRangeSpan( $parser, $text ) {
+   public static function doCharRangeSpan( &$parser, &$text ) {
global $wgCharRangeSpanSettings;
 
foreach( $wgCharRangeSpanSettings as $name => $options ) {
diff --git a/Gruntfile.js b/Gruntfile.js
index 7756e75..a45071e 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -10,7 +10,8 @@
jsonlint: {
all: [
'**/*.json',
-   '!node_modules/**'
+   '!node_modules/**',
+   '!vendor/**'
]
}
} );
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/376774
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7b1d634c9eb52d351d39f57522b2d2e8198f9b73
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/CharRangeSpan
Gerrit-Branch: master
Gerrit-Owner: Umherirrender 
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...CharRangeSpan[master]: Add php-parallel-lint

2017-09-08 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/376774 )

Change subject: Add php-parallel-lint
..

Add php-parallel-lint

php-lint will test for valid php files

Change-Id: I7b1d634c9eb52d351d39f57522b2d2e8198f9b73
---
M .gitignore
M Gruntfile.js
A composer.json
3 files changed, 15 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/CharRangeSpan 
refs/changes/74/376774/1

diff --git a/.gitignore b/.gitignore
index 1689d7a..8ea3b90 100644
--- a/.gitignore
+++ b/.gitignore
@@ -2,3 +2,5 @@
 *.kate-swp
 .*.swp
 node_modules/**
+vendor/**
+composer.lock
diff --git a/Gruntfile.js b/Gruntfile.js
index 7756e75..a45071e 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -10,7 +10,8 @@
jsonlint: {
all: [
'**/*.json',
-   '!node_modules/**'
+   '!node_modules/**',
+   '!vendor/**'
]
}
} );
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/376774
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

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

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