Scott Wegner created BEAM-5742:
----------------------------------

             Summary: Add expectations for code reviews
                 Key: BEAM-5742
                 URL: https://issues.apache.org/jira/browse/BEAM-5742
             Project: Beam
          Issue Type: Sub-task
          Components: website
            Reporter: Scott Wegner


There should be a dedicated section about how we to code reviews, and 
expectations for contributors / reviewers. Things like:
* PR's should have linked JIRA
* Code changes should also include tests
* Small changes are easier to review
* How to find a reviewer, when you can expect reviewer to engage
* How automatic test runs work (PreCommits run by on each commit; build / test 
should be passing before asking for a reviewer)
* How to re-run tests, and how/when to run Post-Commits



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to