Create
cancel
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

Single Master branch and Code reviews

As a recommendation from our QA team, we are maintaining a single master branch in our project and all the team members are checking in to master branch directly. This way the Continuous Integration is simple since it runs on the master and we have all the UT, sanity tools etc. attached to the CI and everything is in place. But the problem is with code reviews. How does the team members raise for code reviews since the everybody checks in frequently to master, for the reviewer it's a challenge to review only one particular authors/feature code. Any alternatives other than having individual branches?

 

2 answers

0 votes

Do you have Crucible too, or just Fisheye? If you have Crucible and you use individual issue keys for every commit, have the change author create a review by adding multiple changesets (pretty easy if you filter by the issue key in the "add content" dialog).

But as Alexey pointed out, if changes are related to the same file, you will get the changes mixed up in the review. The non-review related changes will be indicated in the UI though.

In Crucible, in case of having only master branch, a adding content to review 'Choose branches', 'Add branch' button is disabled. As a workaround I had to use 'Browse changesets', but it doesn't make sense that it's not possible to review via 'Choose branches' at the same time. How to be able to add master branch?

0 votes

If you need to review only single commit then I see no issues here.

But if you need to review several commits mixed with commits from other people, there is no way to divide your changes from other people, especially if you modified the same file. In case you modified different files FishEye allows you to choose commits and review only them

Suggest an answer

Log in or Sign up to answer
TAGS
Community showcase
Published in Confluence Cloud

🎨 Add some visual life to your templates

Hi Atlassian Community, My name is Avni Barman, and I am a Product Manager on the Confluence Cloud team. Based on feedback from you, we are giving admins more power to create templates that a...

224 views 4 9
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