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
@jrfnl has created example code highlighting possible rulesets for plugins and themes (instead of using the full WPCS core ruleset). We should look to support these rulesets for plugin and theme reviews.
Expected Behavior
Add in two new rulesets to support specific reviews for plugins and themes.
Current Behavior
We currently audit plugins and themes against the full WPCS core ruleset and PHP Compat.
Todos
Tests
Documentation
Code
The text was updated successfully, but these errors were encountered:
@grappler in that case, should we split this into two issues (one for plugins and one for themes)? Possibly leaving this one for plugins and creating a new one for themes using details from the ruleset the TRT created?
@jeffpaul Short summary of what was decided regarding this during the meeting at WCEU:
The TRT repo will be renamed to a more generic name and moved to the wp.org GH account.
In the (near) future, the repo will contain:
a Review sniffs directory which should not be directly used, this will (and does) contain theme/plugin review specific sniffs which are too specific for WPCS.
WPThemeReview has some new features, but also has an issue with false positives, especially in escaping and prefixing sniffs. It can be used (it's used in Twenty Twenty theme), but a lot of work needs to be done on it.
Issue Overview
@jrfnl has created example code highlighting possible rulesets for plugins and themes (instead of using the full WPCS core ruleset). We should look to support these rulesets for plugin and theme reviews.
Expected Behavior
Add in two new rulesets to support specific reviews for plugins and themes.
Current Behavior
We currently audit plugins and themes against the full WPCS core ruleset and PHP Compat.
Todos
The text was updated successfully, but these errors were encountered: