Pull requests into Health Data Standards require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Submitter:
[ ] This pull request describes why these changes were made.
[ ] This PR is into the correct branch.
[ ] Internal ticket for this PR:
[ ] Internal ticket links back to this PR
[ ] Code diff has been done and been reviewed (it does not contain: additional white space, not applicable code changes, debug statements, etc.)
[ ] Tests are included and test edge cases
[ ] Tests have been run locally and pass
[ ] Code coverage has not gone down and all code touched or added is covered.
In rare situations, this may not be possible or applicable to a PR. In those situations:
Note why this could not be done or is not applicable here:
Add TODOs in the code noting that it requires a test
Add a JIRA task to add the test and link it here:
Cypress Reviewer:
Name:
[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code
Bonnie Reviewer:
Name:
[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
Pull requests into Health Data Standards require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Submitter:
Cypress Reviewer:
Name:
Bonnie Reviewer:
Name: