VIP’s priority is to ensure that your site is reliable, which means we care about its performance and security. Code review is a key component of ensuring your site is secure and performant.
We encourage you to run the PHP_CodeSniffer tool in your development environment or code editor, allowing you to fix errors as you code and develop to VIP best practices.
When you open a Pull Request (PR) for your codebase in GitHub, we offer both automated scans and manual reviews to clients:
When you open a PR in the GitHub, your entire codebase will be automatically scanned against VIP Coding Standards by the VIP Code Analysis bot. If you have questions about how to address specific errors or warnings, you can open a Zendesk ticket with our team.
Manual code review
For clients with Application Support, you may request specific developer feedback on your code (including themes and custom plugins) by adding the “[VIP] Review Request” label to your PR. Before adding the label, ensure that you’ve addressed as many errors and warnings from the automated scan as possible. If the changeset is larger than 1000 lines of code, it will need to be scheduled for a review. Where possible, we recommend keeping PRs small by breaking them down into atomic commits. Please allow for 10-15 business days in your project timeline to complete the first and subsequent review cycles.
We take code review seriously and understand that there may be many questions along the way. If you need assistance, please open a ticket and we’d be happy to provide guidance.