search

codeclimate / codeclimate-phpcodesniffer

Code Climate Engine for PHP Code Sniffer
MIT License
28 stars 23 forks source link

WordPress-VIP issues display when config is set to "WordPress" #37

Closed sfgarza closed 8 years ago

sfgarza commented 8 years ago

I have my codeclimate config set up like this

phpcodesniffer:
    enabled: true
    config:
      standard: "WordPress"

But WordPress-VIP issues are showing up codeclimate

bhubbard commented 8 years ago

+1

dblandin commented 8 years ago

Hey @sfgarza,

Happy to hear that you're checking out the new Wordpress support!

As it's mentioned in the WordPress-Coding-Standards README, the WordPress standard includes the complete set with all of the sniffs in the project. This includes WordPress-Core, Wordpress-Docs, Wordpress-Extra, and Wordpress-VIP.

If you don't want Wordpress-VIP issues, you should choose another standard option.

This can be a single entry, like "Wordpress-Core", or a comma-separate list, omitting "Wordpress-VIP":

phpcodesniffer:
  enabled: true
    config:
      standard: "WordPress-Core,Wordpress-Docs,Wordpress-Extra"
dblandin commented 8 years ago

Going to close this issue for now. Let me know if you have any more questions. Thanks!

sfgarza commented 8 years ago

Got it. That makes sense. Thanks!