Skip to content

Commit

Permalink
Update PHPCS config (#177)
Browse files Browse the repository at this point in the history
* Rename property

* Update list of ignored file system operations

* Remove redundant excludes

* Undo some changes
  • Loading branch information
swissspidy authored Aug 30, 2023
1 parent 21889bd commit 8dbfadc
Showing 1 changed file with 2 additions and 19 deletions.
21 changes: 2 additions & 19 deletions WP_CLI_CS/ruleset.xml
Original file line number Diff line number Diff line change
Expand Up @@ -79,26 +79,8 @@
<exclude name="Universal.Operators.DisallowShortTernary.Found"/>

<!-- Allow filesystem operations because WordPress APIs may not be available -->
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_chgrp" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_chmod" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_chown" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_fclose" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_file_put_contents" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_fopen" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_fputs" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_fread" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_fsockopen" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_fwrite" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_is_writable" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_is_writeable" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_mkdir" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_pfsockopen" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_readfile" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_rmdir" />
<exclude name="WordPress.WP.AlternativeFunctions.file_system_operations_touch" />
<exclude name="WordPress.WP.AlternativeFunctions.rename_rename" />
<exclude name="WordPress.WP.AlternativeFunctions.unlink_unlink" />

</rule>


Expand All @@ -125,7 +107,7 @@
See: https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/wiki/Customizable-sniff-properties#error-silencing-use-build-in-function-whitelist -->
<rule ref="WordPress.PHP.NoSilencedErrors">
<properties>
<property name="use_default_whitelist" value="true"/>
<property name="usePHPFunctionsList" value="true"/>
</properties>
</rule>

Expand All @@ -145,6 +127,7 @@
<property name="exclude" type="array">
<element value="curl"/>
<element value="file_get_contents"/>
<!-- Allow filesystem operations because WordPress APIs may not be available -->
<element value="file_system_operations"/>
<!-- As PHP 5.4. is the minimum for most projects, using json_encode() is fine. -->
<element value="json_encode"/>
Expand Down

0 comments on commit 8dbfadc

Please sign in to comment.