Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Code review of a branch in GitHub

Tags:

git

branch

github

I need to do a code review of a whole branch in GitHub. I went to the branches page and hit compare. There's a tab there, files changed, that shows exactly what I need, the diff between the two branches, master and the selected one. The problem is that it doesn't allow you to comment on code like a single diff does:

github code review interface

The blue bubble just doesn't appear on the branch diff page.

Does anybody know how to perform a code review of a whole branch in GitHub?

Thanks.

like image 704
pupeno Avatar asked Aug 26 '11 07:08

pupeno


People also ask

How do I review a branch on GitHub?

I suggest to start with the branch name review/<review_subject> in order to easily distinguish it from regular branches. Push the branch. Create a new pull request on GitHub. Pull the branch that you want to be reviewed into the review branch.

How does GitHub approve code review?

Above the changed code, click Review changes. Type a comment summarizing your feedback on the proposed changes. Select Approve to approve merging the changes proposed in the pull request. Click Submit review.

Is pull request same as code review?

Open a Pull Request – This is a formal request for your commit to be reviewed by another developer. Code Review – A developer agrees to perform a code review on your commit, which may be done formally or informally. If approved, your commit takes a step forward (to #5 or #6 depending on your team's process).


1 Answers

Apparently there's no UI to write comments on a branch diff. My solution for now was asking the other programmer to do a pull request. It did work to do a pull request from one project to itself and in the pull request I can write comments.

like image 134
pupeno Avatar answered Sep 28 '22 08:09

pupeno