Skip to content

Latest commit

 

History

History
50 lines (40 loc) · 2.18 KB

pull_request_template.md

File metadata and controls

50 lines (40 loc) · 2.18 KB

Summary

Author checklist

  • Tutorial for replace_with_issue_numbers
  • Has the new tutorial been integrated into the main level? Yes/No
  • Has a new section of the readme been added for this tutorial? Yes/No
  • Have you done a full self-review of the code and blueprints? Yes/No
  • Have you tagged a primary reviewer? Yes/No
  • Are there any remaining tasks to do or blocking questions to answer before we can merge this? Yes/No

Testing plan

Reminders for reviewers

Thank you for taking the time to review this PR. By approving a PR you are taking as much responsibility for these changes as the author. Please keep these points in mind throughout the review process:

  • Review and run all parts of the test plan on this branch and verify it matches expectations.
  • Review the code and make sure you do not have any remaining questions or concerns. You should understand why this code change fixes the issue and agree that it's the best approach to doing so.
  • Review the automated tests and make sure there are no missing tests or edge cases.
  • When you feel this is ready bump it to a second reviewer or merge.