Alpha Release Review Workflow
Library Module Alpha PR Review Workflow
This describes the process for reviewing a PR to a library module dev branch.
Application
This process applies when reviewing a PR to the dev branch of a library module.
Process
-
Complete the following. Note any issues in Review Comments before leaving a review to
Request Changes. If there are no comments or the comments do not require code changes, then you mayApprovethe PR.- Review changes to imports and verify dependency minimum versions are accurate.
- Ensure code changes have adequate unit test coverage.
- Ensure there are no breaking changes.
- If changes replace or could replace an existing method, ensure a deprecation
warning is logged in the replaced method (either
deprecateddecorator orlog_deprecationcall). - If tests are modified, make sure newly deprecated methods are still tested and that changes do not remove any test cases.
- Highlight any added
TODOor existingTODOcomments related to the changes.It is up to the PR author and reviewer to decide if a `TODO` must be completed before merging OR if an issue can be created to address it in a later PR.
-
Ensure the PR title and body are completed. Make any revisions necessary or work with the PR author to make those changes.
- Leave a review to
ApproveorRequest Changes. If reviewing a draft or if unable to test changes, leave aCommentreview indicating why it isn't an Approval, i.e. "Unable to test." or "Noted pending TODO"
Exceptions
- There should be no exceptions to this review process.