- Test your PR from scratch locally: bootstrap, setup and test the part of the interface touched by your work
- Create the PR in Draft mode
- Make sure your branch is up-to-date with the desired branch (parent branch you come from)
- Make sure you do not have extra commits to squash
- Review your own PR first
- If an LLM helped you write the code, you MUST have read and understood what you commit BEFORE calling other reviewers!
- Update the commit message after the review to be sure that all what you have done is described
- Check that the Github Actions tests are successfull
- Remove the draft mode
- Wait for CodeRabbit to review your PR, and make any fixes necessary
- Call 1-2 pertinent team members for review
- For large code changes (100+ lines) or when you are not entirely sure about your changes, speak with the team about what you did and what they should pay attention to.
- If you need PO tests, ask for local testing or deploy to a dev server.
- When your PR is approved and tested, do a rebase and merge.