Try dotcom's set of sniffs
This commit is contained in:
parent
3eab03225b
commit
d3b65f255c
2 changed files with 52 additions and 1 deletions
|
@ -13,7 +13,9 @@
|
||||||
"squizlabs/php_codesniffer": "3.*",
|
"squizlabs/php_codesniffer": "3.*",
|
||||||
"wp-coding-standards/wpcs": "*",
|
"wp-coding-standards/wpcs": "*",
|
||||||
"yoast/phpunit-polyfills": "^2.0",
|
"yoast/phpunit-polyfills": "^2.0",
|
||||||
"dealerdirect/phpcodesniffer-composer-installer": "^1.0.0"
|
"dealerdirect/phpcodesniffer-composer-installer": "^1.0.0",
|
||||||
|
"sirbrillig/phpcs-variable-analysis": "^2.11",
|
||||||
|
"phpcsstandards/phpcsextra": "^1.1.0"
|
||||||
},
|
},
|
||||||
"config": {
|
"config": {
|
||||||
"allow-plugins": true
|
"allow-plugins": true
|
||||||
|
@ -36,6 +38,9 @@
|
||||||
"composer install",
|
"composer install",
|
||||||
"bin/install-wp-tests.sh activitypub-test root activitypub-test test-db latest true",
|
"bin/install-wp-tests.sh activitypub-test root activitypub-test test-db latest true",
|
||||||
"vendor/bin/phpunit"
|
"vendor/bin/phpunit"
|
||||||
|
],
|
||||||
|
"lint": [
|
||||||
|
"vendor/bin/phpcs"
|
||||||
]
|
]
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
46
phpcs.xml
46
phpcs.xml
|
@ -23,4 +23,50 @@
|
||||||
<rule ref="WordPress-Extra" />
|
<rule ref="WordPress-Extra" />
|
||||||
<rule ref="WordPress.WP.I18n"/>
|
<rule ref="WordPress.WP.I18n"/>
|
||||||
<config name="text_domain" value="activitypub,default"/>
|
<config name="text_domain" value="activitypub,default"/>
|
||||||
|
<arg value="ps"/>
|
||||||
|
<arg name="parallel" value="20"/>
|
||||||
|
<rule ref="VariableAnalysis"/>
|
||||||
|
<rule ref="VariableAnalysis.CodeAnalysis.VariableAnalysis.UndefinedVariable">
|
||||||
|
<type>error</type>
|
||||||
|
</rule>
|
||||||
|
<rule ref="VariableAnalysis.CodeAnalysis.VariableAnalysis">
|
||||||
|
<properties>
|
||||||
|
<property name="allowUnusedCaughtExceptions" value="true"/>
|
||||||
|
</properties>
|
||||||
|
</rule>
|
||||||
|
<rule ref="WordPress"/>
|
||||||
|
<rule ref="WordPress.WP.I18n.NoHtmlWrappedStrings">
|
||||||
|
<type>error</type>
|
||||||
|
</rule>
|
||||||
|
<rule ref="Generic.CodeAnalysis.UnusedFunctionParameter"/>
|
||||||
|
<rule ref="Generic.Arrays.DisallowShortArraySyntax">
|
||||||
|
<severity>0</severity>
|
||||||
|
</rule>
|
||||||
|
<rule ref="Universal.Arrays.DisallowShortArraySyntax">
|
||||||
|
<severity>0</severity>
|
||||||
|
</rule>
|
||||||
|
<rule ref="Squiz.Commenting">
|
||||||
|
<severity>0</severity>
|
||||||
|
</rule>
|
||||||
|
<rule ref="Generic.Commenting">
|
||||||
|
<severity>0</severity>
|
||||||
|
</rule>
|
||||||
|
<rule ref="WordPress.Files.FileName">
|
||||||
|
<severity>0</severity>
|
||||||
|
</rule>
|
||||||
|
<rule ref="WordPress.DB.PreparedSQL.NotPrepared">
|
||||||
|
<severity>0</severity>
|
||||||
|
</rule>
|
||||||
|
<rule ref="WordPress.WP.CapitalPDangit">
|
||||||
|
<severity>0</severity>
|
||||||
|
</rule>
|
||||||
|
<rule ref="WordPress.Arrays.ArrayDeclarationSpacing.AssociativeArrayFound">
|
||||||
|
<severity>0</severity>
|
||||||
|
</rule>
|
||||||
|
<rule ref="WordPress.PHP.YodaConditions.NotYoda">
|
||||||
|
<type>warning</type>
|
||||||
|
</rule>
|
||||||
|
<rule ref="WordPress.Arrays.ArrayDeclarationSpacing">
|
||||||
|
<exclude-pattern>**/*.asset.php</exclude-pattern>
|
||||||
|
</rule>
|
||||||
</ruleset>
|
</ruleset>
|
||||||
|
|
Loading…
Reference in a new issue