Showing results for 
Search instead for 
Did you mean: 
Sign up Log in
It's not the same without you

Join the community to find out what other Atlassian users are discussing, debating and creating.

Atlassian Community Hero Image Collage

How to select git commits for a review of a feature branch with updates?

We have a workflow where devs do feature branches, and update from (that is, merge in) the master branch occasionally. [Rebases are not an option.]

So, we have two states A and B on the master branch. dev starts its branch off A, does work in F and G, then updates from the later master commit B resulting in merge commit M, and finally does some more stuff as P. [See pic (linked) below]

Now, what commits would I select when creating a crucible review. Then (apparently naive) guess would be F, G, P, as those contain the work the dev did. But the resulting overall diff is the same as with F, G, M, P - it includes all the new features that got merged in in the update. Not what I want.

The original F,G,P selection does not even allow me to review the change of P separately - the commits available in the selection bead line are only A,F,G,P, so I can only see the diff from G to P which puts together the merge differences (M) and the dev's last change (P).

As the total change I want to review is B...P I finally try to give M and P, but stupidly in this case crucible is using the first parent of M (which is G) as the starting commit of the review, not B, so my review offers no possibility to see the effective diff the dev did with his work.

So basically there seems no way to create a single (or even a set of) review(s) that can show exactly the changes the dev created - it always suffer from the flattening of the actual git commit graph into a single bead line (the red/gray thing that allows review range selecton) plus the fact that a review is not defined by start and end point but rather by naming a set of commits.

After writing that discussion my expectation is that the commit bead line shouldn't be a line but indeed a graph, and the default pair of commits to diff over should be B..P, so I can see the full dev's diff, and also watch (and discuss) intermediate changes he did.

tl;dr crucible should take into account the branching structure of git, and especially it should allow for merge commits to select the parent against which to diff.

So, how do I work around this?

Branch graph:dag (

1 answer

Second one of these I find today. With NO answer!

Suggest an answer

Log in or Sign up to answer
Community showcase
Published in Apps & Integrations

How we use Jira Service Management for our recruitment process! - Part 2

It is never about setting up a process and being done with it. Rather, the focus should always be on optimizing it for the best outcomes. Thus, we didn’t stop at setting up JSM for our recruitment pr...

187 views 0 5
Read article

Community Events

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

Find an event

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

Unfortunately there are no Community Events near you at the moment.

Host an event

You're one step closer to meeting fellow Atlassian users at your local event. Learn more about Community Events

Events near you