Reduce review times
Most pull requests can be reviewed and approved more quickly by following these guidelines.
Before creating a pull request—or requesting a VIP review for a pull request
- Scan the updated code with PHP Code Sniffer using the VIP Coding Standards and address any issues reported in the feedback.
- Get reviews and feedback from other internal team members.
- Create smaller, separate pull requests. Especially for changes that affect more than 1000 lines of code.
- Separate pull requests should be created for:
- Extensive whitespace changes (e.g., fixing tabs and spacing)
- Changes that are limited to style only (e.g., revising variable names, formatting, comment blocks, etc)
- Addition of a plugin: a separate pull request should be created for each plugin added.
- Removal of a plugin, or removal of several files.
- A build step that results in minified or compiled JS and CSS. Setting up automated build and deploy branches may useful to simplify this process.
- Remove unused or unnecessary code from the
master
branch that does not need to be reviewed. - Commit messages in a pull request and the initial comment are helpful to VIP. Summarize the intent of the changes and add detail about complex abstractions.
- If a pull request is related to an open VIP Support ticket, provide a link to that ticket in the comment.
After a pull request is created
- View the pull request in GitHub and confirm that only intended changes are included, especially if there are changes made to
.gitignore
. - Pull requests that contain only CSS changes (or do not contain any files with extensions that we review, such as PHP and JS) will be auto-approved.
- Development related to the pull request should be complete, with the exception of changes made to address feedback from the VIP Code Analysis Bot and the VIP reviewer’s comments.
- When making changes in response to a review, minimize the number of new commits in order to simplify the VIP reviews needed.
- Avoid making changes to an approved pull request. If possible, make needed changes in another branch, or wait until after the open pull request is merged.