How do you do code review for a large commit

When a developer pushes a large commit, or a story has several commits which has lots of files to review, it's very to review it in any code review tool. The best way should be in the IDE, which enables you to easily jump from one method call to its implementation which is another class. etc

How do you tackle this problem?

1 answer

0 vote
Nick Pellow Atlassian Team Dec 10, 2013

I simply leave the review open in my web browser, then drill around, test, (even sometimes spike refactoring suggestions) etc in my IDE, and each time I find something I want to comment on, just CMD-TAB back to the code review and leave a comment.

I spend a lot more time reading and playing with the code for a code review, than leaving a comment, so this setup works well for me.

Suggest an answer

Log in or Join to answer
Community showcase
Davin Studer
Published Thursday in Confluence

FUME – A Better Confluence User Macro Editor Experience

...\\.title', editorDiv: 'userMacroEditorDiv', isNew: function(){ return AJS.$('#user-macro-form').prop('name') == 'addusermacro' ? true : false; }, editor: null, isDirty...

45 views 0 4
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
Atlassian Team Tour

Join us on the Team Tour

We're bringing product updates and pro tips on teamwork to ten cities around the world.

Save your spot