Merge pull request #108 from wmde/wbCs

Use Wikibase's CodeSniffer instead of MediaWiki's

Author: Ladsgroup

composer.json

@@ -29,7 +29,7 @@
 		"composer/installers": ">=1.0.1"
 	},
 	"require-dev": {
-		"mediawiki/mediawiki-codesniffer": "~0.5"
+		"wikibase/wikibase-codesniffer": "^0.1.0"
 	},
 	"autoload": {
 		"files" : [
@@ -38,7 +38,7 @@
 	},
 	"scripts": {
 		"phpcs": [
-			"vendor/bin/phpcs lib/* src/* --standard=phpcs.xml -sp"
+			"vendor/bin/phpcs . -sp"
 		]
 	}
 }

phpcs.xml

@@ -1,38 +1,6 @@
 <?xml version="1.0"?>
 <ruleset name="DataValuesJavaScript">
-	<!-- See https://github.com/wikimedia/mediawiki-tools-codesniffer/blob/master/MediaWiki/ruleset.xml -->
-	<rule ref="vendor/mediawiki/mediawiki-codesniffer/MediaWiki">
+	<rule ref="vendor/wikibase/wikibase-codesniffer/Wikibase">
 		<exclude name="Generic.Arrays.DisallowLongArraySyntax" />
 	</rule>
-
-	<rule ref="Generic.ControlStructures" />
-	<rule ref="Generic.Files.LineLength">
-		<properties>
-			<property name="lineLimit" value="124" />
-		</properties>
-	</rule>
-	<rule ref="Generic.PHP.CharacterBeforePHPOpeningTag" />
-
-	<rule ref="PSR1" />
-	<rule ref="PSR1.Methods.CamelCapsMethodName.NotCamelCaps">
-		<!-- Exclude test methods like "testGivenInvalidInput_methodThrowsException". -->
-		<exclude-pattern>tests*Test*\.php</exclude-pattern>
-	</rule>
-
-	<rule ref="PSR2.Files" />
-
-	<rule ref="Squiz.Classes.DuplicateProperty" />
-	<rule ref="Squiz.Classes.SelfMemberReference" />
-	<rule ref="Squiz.ControlStructures.ControlSignature" />
-	<rule ref="Squiz.Functions.FunctionDuplicateArgument" />
-	<rule ref="Squiz.Functions.GlobalFunction" />
-	<rule ref="Squiz.Scope" />
-	<rule ref="Squiz.WhiteSpace.FunctionSpacing">
-		<properties>
-			<property name="spacing" value="1" />
-		</properties>
-	</rule>
-
-	<arg name="extensions" value="php" />
-	<arg name="ignore" value="*\.js" />
 </ruleset>