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

Already on GitHub? Sign in to your account

PHP_CodeSniffer Code Standards #70

Merged
merged 1 commit into from Nov 28, 2011

Conversation

Projects
None yet
1 participant
Owner

GaryJones commented Nov 28, 2011

Having run PHP_CodeSniffer with the WordPress CS (falls back to using PEAR standards when WP CS isn't explicit) on the code, I found 112 errors. Most fall under the control structures category - including a blank line within the start or end of { } for if, foreach, etc.
This commit addresses all but 1 error, and this is where we don't assign the `new TGM_Plugin_Activation;! instantiation to a (global) variable, which I'm fine with at least for now.

@GaryJones GaryJones added a commit that referenced this pull request Nov 28, 2011

@GaryJones GaryJones Merge pull request #70 from GaryJones/dev
PHP_CodeSniffer Code Standards
fa7736d

@GaryJones GaryJones merged commit fa7736d into TGMPA:dev Nov 28, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment