Update phpcs.xml.dist and fix warnings - Githubissues

studiopress / genesis-sample

This is the sample theme created for the Genesis Framework.

https://demo.studiopress.com/genesis-sample/

528 stars 284 forks source link

Update phpcs.xml.dist and fix warnings #320

Closed dreamwhisper closed 4 years ago

dreamwhisper commented 4 years ago

WPCS 2.2.0 issues warnings for short array syntax usage. These are intended for WordPress core and are safe to ignore. This PR excludes short array syntax warnings from our phpcs standards checks.

It also ignores meta_key and meta_value warnings, which we depend on for the WooCommerce code.

To test

Delete composer.lock (ensures the latest version of wpcs is installed in the third step, but composer update usually works too).
Run rm -rf vendor from the theme directory.
Run composer install and then composer phpcs

There should be no warnings.