You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Instead of ignoring test files and allowing code to be written in any format (https://github.com/alleyinteractive/create-wordpress-project/blob/develop/.phpcs.xml#L30) we should relax the standards for tests but still allow them to be reviewed by PHPCS. The suggestion is to adjust the standards to ensure that tests follow the WordPress coding style but aren't subject to the highest standards. This would mean relaxing the more dangerous sniffs such as direct database queries, meta queries, etc.
Acceptance Criteria
Tests can be included in PHPCS configurations and will not be excluded by create-wordpress-project/create-wordpress-plugin.
Tests won't be subject to many sniffs that don't apply to unit tests (direct database calls, meta queries, etc.).
Tests are written in this repository to enforce this.
The text was updated successfully, but these errors were encountered:
Instead of ignoring test files and allowing code to be written in any format (https://github.com/alleyinteractive/create-wordpress-project/blob/develop/.phpcs.xml#L30) we should relax the standards for tests but still allow them to be reviewed by PHPCS. The suggestion is to adjust the standards to ensure that tests follow the WordPress coding style but aren't subject to the highest standards. This would mean relaxing the more dangerous sniffs such as direct database queries, meta queries, etc.
Acceptance Criteria
create-wordpress-project
/create-wordpress-plugin
.The text was updated successfully, but these errors were encountered: