Search code examples
perforcereview-board

How to avoid review board interfering with our workflow?


My team is starting to use Review Board with Perforce with a pre-commit review. We want to send all pre-commits to reviewboard. Though I believe pre-commit is the best way to go about this (than a post-commit review), I have some questions:

  1. How does someone verify their changes work with the build system before committing the change? I would want to be confident that the changes I'm submitting for review will build.
  2. Some people like to make small commits. They could be to the same file(s). How do we balance making small commits with submitting each change pre-commit to review board?

Solution

  • 1) For the first question, if you're using Perforce, one answer is to build shelved changes, like the answer here: https://stackoverflow.com/a/34881362/369243

    2) Perforce is not a good solution for making small commits, but git is. There can be multiple small commits in a merge request.