Engineering philosophy
Summarize your team's approach to engineering here.
Preparing Code for Review
Preparation sets your reviewers up for success.
Commit Messages
Make sure your commit messages are descriptive.
Github PR Descriptions
Your PR descriptions should be an extension of your commit messages. Write about both what the commit changes, and how you implemented the change.
Code Review Guidelines
-
Code review is a priority
-
PR reviews by 2 devs
-
Not more than 2 open PRs
-
Code review rules
- Ensure PRs are good, i.e. short and well labeled
- Use relevant commit messages and descriptions in the PR
- Automate as much as possible
- Set CI/CD pipelines
- Set code style formatters and linters
- Deploy branch to test environment
-
Recommended to squash all commits to one commit when sending PRs.
-
If your PR is pending some other PR then you can do these to unblock yourself:
- Pull the changes from that branch to your branch locally and later raise PR against master once the parent PR is merged.
- Create a sub-branch from that branch and pull change. Later you raise PR against master once the parent PR is merged.
- Make two passes over the PR if it's substantial.
- On the first pass, come to an understanding of the code change at a high level.