workflow requiring code review

I'm looking to make code reviews a required step to accept code into our code base. Been playing with Stash & Crucible to see how they could be setup to help me accomplish this, but am struggling to find a good answer.

To start with, I think the basic features I'm looking for (with the!workflow-feature-branch in mind) are:

  1. Require review approval to merge code on the "master/develop" branch. Note: various options for submission here (patches, feature branches to merge, forks), I'd prefer to allow submitters to work/push "feature" branches and control merges into restricted branches.
  2. Require a minimum of X approvals (based on the repo/repo area)
  3. Required approvers come from a pre-defined list of users (based on the repo/repo area)
  4. Self-approval toggle switch (e.g. can one self-approve or not?)
  5. Allow additional reviewers to participate

Stash, via the "pull request" system seems to cover only #2 (and #5?), but not the rest, although it's pretty close on #1 as it has ACLs for branches, but not based on pull request status.

Crucible, surprisingly seems to fall short on all fronts. (uh?!) What am I missing here?

Curious about whether I'm looking at this the wrong way, or whether there are plugins/hooks that'd do the job here.

1 answer

Hi Chris,

How did you go?

1. This isn't currently possible without writing your own plugin. You might be interested in the following:

2. Yep.

3. Again you could do this with a plugin. There are quite a few open feature requests around reviewers, but nothing that matches that exact requirement. You might want to do a quick Google if you're interested. See below also.

4. People with write access should be able to approve without being a reviewer. Is that what you had in mind?

5. Again, people with access should be able to add themselves or others to a pull request.

In general we're planning on enhancing the whole pull request experience in feature releases. I can't be more specific around dates or exact features, but we'll certainly be keeping 1 and 3 in mind.



Suggest an answer

Log in or Sign up to answer
Community showcase
Published Nov 06, 2018 in Bitbucket

Upgrade Best Practices

Hello! My name is Mark Askew and I am a Premier Support Engineer for products Bitbucket Server/Data Center, Fisheye & Crucible. Today, I want to bring the discussion that Jennifer, Matt, and ...

1,906 views 7 10
Read article

Atlassian User Groups

Connect with like-minded Atlassian users at free events near you!

Find a group

Connect with like-minded Atlassian users at free events near you!

Find my local user group

Unfortunately there are no AUG chapters near you at the moment.

Start an AUG

You're one step closer to meeting fellow Atlassian users at your local meet up. Learn more about AUGs

Groups near you