It seems to be prevented by default for clear reasons, but we require this for what I believe to be a common scenario:
- All code needs to be reviewed by someone else than the author
- More than one person can commit to a branch/PR
- If two people author code to a branch, it should be enough for those same two people to review/approve the PR. This way all code is reviewed by at least one person other than the author.
Enabling a workflow described above requires two things from Stash:
- Set minimum required amount of approvers for the repoitory to 2 persons (already works!)
- Allow the creator of the PR to be added as a reviewer (currently prevented by Stash)
The "downside" would be that the author of a PR would always need to approve the branch, despite the branch containing commits from a single author. This is really just a minor annoyance, though, as it solves a much bigger problem.