Code review is integral to the development of high-quality software. All Assembla version control repositories come with lightweight code review tools built-in. Simply add a comment to a commit under the Commits sub-tab or start a merge request to begin a code review.
Using merge requests as code reviews
Merge requests (MR) are the most common and recommended way to facilitate code review in Assembla. It is a request from someone to merge in code from one branch into another. You can create an MR from Assembla by clicking New Merge Request from Git, SVN, or Perforce repository tools in the Assembla web app.
You’ll be asked to select a From branch and a To branch. Once you do this, you’ll see that the Create Merge Request button is enabled, but only if there are changes between the two branches. You'll also see some initial information about the changes, such as commits, files affected, and tickets affected.
Reviewing code via merge requests
Once you create a merge request, you can:
View information about the MR, including source branch, destination branch, create date, MR id, status, and instructions to update your local environment with the changes included in this MR.
Add followers to request their review.
@Mention teammates in the comments to let them know the MR is ready for them to review.
Add inline comments under the Files tab to make specific suggestions about changes to the code with file attachments, code snippets, screenshots and more.
Reply to other teammates' comments to keep distinct conversation threads together.
Mark comments as Resolved once requested changes are made.
View a list of the commits included in the MR as well as the history of different versions of the MR over time.
Scroll through all the activity on the MR, including comments, commits, and votes.
Vote the current version of the MR up or down.
Jump to Assembla tickets associated with the MR or any of the included commits.
And ultimately merge the changes into the destination branch once your team has approved.
Getting the most out of Assembla merge requests
Set a policy for the number of reviewers and upvotes required by your team to approve merge requests. Adding new commits to the From branch of your merge request automatically resets the vote counts so that your team can re-review your updated changes.
Integrate Assembla merge requests with your CI/CD tools via our built-in Jenkins integration (Git repos only) or Assembla API to show whether the changes pass or fail.
Set up protections on Git branches or Subversion (SVN) directories to control which team members need to ultimately approve and merge requested changes into the master branch or trunk.
Mark MR comments as "Resolved" once requested changes have been made as a handy indicator of what was done and what is left to do.
Associate Assembla Tickets with your merge request by referencing them in the MR description box ("re #1"), by manually adding them on the right-hand side of the request after you have created it, or searching for the merge request on the left-hand side of any ticket view. This allows you to provide quick links to any of the tickets associated with the changes included in your work and the status of connected merge requests from your Assembla tickets.
Need help? Contact us at support@assembla.com.