darrenjacoby / intervention

WordPress plugin to configure wp-admin and application state using a single config file.
MIT License
639 stars 51 forks source link

Add support for WordPress code style/format/linting #107

Closed claytoncollie closed 2 years ago

claytoncollie commented 2 years ago

I opened the plugin files up in my VS Code and noticed a ton of formatting issues. I have my machine set to use the WordPress-Extra standard when writing code. Would you be open to linting this entire plugin against a standard? We can have it run against all pull requests and all releases/deployments as well. I would be happy to work on this although it would be a huge PR (could be in pieces) since it will probably touch every single file. This is a nice article to get you going with PHPCS. Sorry if you already know about/how to do this 😄

claytoncollie commented 2 years ago

Sorry, just now seeing this is the standard symplify/easy-coding-standard

darrenjacoby commented 2 years ago

@claytoncollie I'm open to review it actually, symplify/easy-coding-standard was a contribution. I'm not a fan of the WordPress coding standards, but open to implementing PSR-12.

darrenjacoby commented 2 years ago

Closing for now.