Skip to content

Latest commit

 

History

History
30 lines (28 loc) · 1.4 KB

pr-template.md

File metadata and controls

30 lines (28 loc) · 1.4 KB

What are the links to relevant tickets?

What does this PR do?

What functional/unit tests does this PR have?

Is there anything a developer should be aware of when reviewing this?

Screenshots (if appropriate)


Quality checklist (for PR author to complete BEFORE code review):

  • I've checked there is appropriate unit test coverage.
  • I've updated documentation with any changes to procedures.
  • I've checked this work against the requirements of the Jira.
  • This change passes all necessary tests (cross browser, automated or otherwise).
  • I am ready for this to be code reviewed, merged and tested.

Reviewer 1

  • I agree with the assumptions made in the quality checklist.
  • I’ve witnessed the work behaving as expected.
  • I’ve checked for appropriate test coverage.
  • I’ve checked for coding anti-patterns.
  • I've checked this work against the requirements of the Jira.
  • I don’t believe this work introduces unnecessary technical debt.
  • +1 from me!

Reviewer 2

  • I agree with the assumptions made in the quality checklist.
  • I’ve witnessed the work behaving as expected.
  • I’ve checked for appropriate test coverage.
  • I’ve checked for coding anti-patterns.
  • I've checked this work against the requirements of the Jira.
  • I don’t believe this work introduces unnecessary technical debt.
  • +1 from me!