Skip to content

Latest commit

 

History

History
8 lines (8 loc) · 711 Bytes

Expectations_of_Code_Review.md

File metadata and controls

8 lines (8 loc) · 711 Bytes

Reviewer Expectations for Code Review

  • Have verified that Definition of Done has been completed
  • Visual inspection of code to see that it does what it needs to and doesn't do what is not reported
  • Confirmed that all comments and questions raised in discussion of the pull request have been resolved
  • If possible, a reviewer should build and test the code changes
  • Merging a pull request is a statement of "I believe this code is of good quality"
  • Check that consensus voting has resulted in approval of each issue resolved with the pull request (see governance document for details)
  • Check that consensus voting has resulted in approval of the pull request itself (see governance document for details)