A peer review is a necessary confirmation that the submitted code is of sufficiently high quality so as to be eligible for inclusion in the OSEHRA code base. Peer reviews can be made by anyone, and multiple peer reviews are allowed and encouraged; however, at least one passing peer review must be made by a trusted individual if the code is to be considered for adoption.
Gerrit peer review is based on the “OSEHRA Peer Review Checklist”. This should be downloaded from the OSEHRA web site, http://www.osehra.org prior to beginning the review process.
To perform a peer review, go to the OSERHA Gerrit review site at http://review.code.osehra.org and log in. Find the article you want to review. Click on the article to bring up the publication page, and then click the one of the diff buttons (blue arrows) to bring up a code review tool (Figure 3). Verify that the code looks to fix the corresponding Jira issue and that the code is appears to be compliant with the OSEHRA SAC. Walk through the “OSEHRA Peer Review Checklist” executing all the appropriate tests for Safe, Compliant, and Functional and marking all items Pass or Fail. Once the status of the code with respect to the checklist has been determined, press the review button to bring up the attestation page (Figure 4). For each of Safe, Compliant, and Functional; mark the section +1 if all the items in the checklist have a pass for the section. If all three sections have a pass and the visual code review looked good, mark the Code Review Attestation +1 (+2 if you are a trusted reviewer). Press Publish Comments to end the review and submit the results to the OSEHRA Dashboard. Note that any sections marked -1 when the “Submit Review” button is clicked will show up as a failing test on the dashboard. Finally complete the review by uploading the completed checklist document to the Jira issue tracker, attaching it to the corresponding Jira issue.
Figure 3- Gerrit Code Review Site with Review Button (red arrow) and Diff Buttons (blue arrows).
Figure 4- Code review attestations.