Commit graph

56 commits

Author SHA1 Message Date
Roan Kattouw cf1da49904 Remove forced -p (progress) flag from phpcs.xml
This causes phpcs to emit invalid JSON when the --report=json
flag is set and no issues are found, which breaks editor plugins
(atom-linter-phpcs in my case).

Add this flag in composer.json instead.

Change-Id: Ibb0b4c084e25e06be72389f1d4998804cfd53295
2016-04-26 14:21:28 -07:00
Bryan Davis 3f7436c959 Wrap CallbackFilterIterator in a condition check
Wrap the CallbackFilterIterator backport class in a conditional check
for PHP runtimes that include the class natively. This really should
only be needed for linting as the class is loaded via an autoloader
and thus should not be loaded if the runtime already has it
available.

Bug: T124828
Change-Id: I39d27385186d4693a8babdd2b818e6b4bc16255a
2016-01-26 14:57:18 -07:00
Kunal Mehta c07236a121 build: Updating development dependencies
* jakub-onderka/php-parallel-lint: 0.9 → 0.9.2

Change-Id: I8dea1aa0be23a7de6e784ed4fd60397b7294f0ea
2016-01-05 10:34:05 -08:00
Kunal Mehta 95cbd5de30 build: Updating development dependencies
* mediawiki/mediawiki-codesniffer: 0.5.0 → 0.5.1

Change-Id: I46a926adc0b2d00058ef732bd39037a46e89ebf2
2016-01-03 18:47:18 -08:00
Kunal Mehta cb82edfe94 Run phpcs through "composer test", disabling failing rules
Disabling:
* Generic.Files.LineLength
* Generic.Functions.FunctionCallArgumentSpacing.SpaceBeforeComma
* Generic.PHP.ForbiddenFunctions.FoundWithAlternative
* MediaWiki.NamingConventions.PrefixedGlobalFunctions.wfPrefix
* MediaWiki.Usage.DirUsage.FunctionFound
* MediaWiki.VariableAnalysis.UnusedGlobalVariables
* MediaWiki.WhiteSpace.SpaceAfterControlStructure.Incorrect
* MediaWiki.WhiteSpace.SpaceBeforeSingleLineComment
* PSR2.Classes.PropertyDeclaration.ScopeMissing
* PSR2.Classes.PropertyDeclaration.VarUsed
* PSR2.Methods.MethodDeclaration.AbstractAfterVisibility
* PSR2.Methods.MethodDeclaration.StaticBeforeVisibility
* Squiz.Classes.ValidClassName.NotCamelCaps
* Squiz.WhiteSpace.LanguageConstructSpacing.Incorrect

Disabled rules will be enabled individually in follow-up patches.

Bug: T101074
Change-Id: I1d52d891eb8e4f5877d1f1f3fa3aa9fc28dff004
2015-10-29 12:52:32 +01:00
Paladox 3dbe5bb0ec Add composer.json
This add php parallel lint test. Can be triggered using the entry
point:

 composer test

That ease Jenkins integration and let developers add more testing
utilities as needed.

Change-Id: I2bf0dbea9f559267001eb727c7ecfeb103717464
2015-10-13 09:48:03 +00:00