Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Are the correct rulesets being used ? #16

Closed
jrfnl opened this issue Dec 22, 2016 · 2 comments
Closed

Are the correct rulesets being used ? #16

jrfnl opened this issue Dec 22, 2016 · 2 comments
Assignees
Milestone

Comments

@jrfnl
Copy link
Collaborator

jrfnl commented Dec 22, 2016

WPCS currently has five rulesets:

  • WordPress-Core - stand-alone, only rules as documented for core
  • WordPress-Docs - stand-alone, only documentation rules as documented for core
  • WordPress-Extra - WordPress-Core + a number of extra programming/WP best practice kind of rules
  • WordPress-VIP - WordPress-Core + a number of specific rules which plugins and themes have to comply with to be allowed on to the WP VIP platform.
  • WordPress - WordPress-Core + WordPress-Docs + WordPress-Extra + WordPress-VIP (all current rulesets)

Currently Yoastcs includes the WordPress-VIP ruleset and the WordPress-Docs ruleset.

As far as I know, the Yoast plugins are not deployed on the VIP platform, so do not have to comply with the VIP rules - though they do contain some hints/best practices which may be useful to check against.

So having said that, to me, it would seem more logical to check against either WordPress (everything) or against WordPress-Extra + WordPress-Docs.

Opinions ?

@atimmer
Copy link
Contributor

atimmer commented Dec 23, 2016

I would prefer checking against WordPress

jrfnl added a commit that referenced this issue Dec 23, 2016
@jrfnl jrfnl self-assigned this Dec 23, 2016
@jrfnl
Copy link
Collaborator Author

jrfnl commented Dec 23, 2016

See PR #19.

I suggested Extra + Docs as an alternative as the Yoast-CS ruleset currently contains quite a few exclusions for VIP specific rules, so it would be worth reviewing what's left of the VIP ruleset after all the exclusions and if it's worth keeping.

@jrfnl jrfnl added this to the 0.4.x milestone Nov 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants