chore: adds reviewer checklist to PR template

[jpower432]

Description

Adds reviewer checklist

Type of change

• This change requires a documentation update

How has this been tested?

N/A

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[jpower432]

@afflom @beatrizmcouto This is still WIP, but hoping to use the PR as a jumping off point for us to collaborate on author and reviewer requirements before merge. Let me know what you think of the initial checklists.

[beatrizmcouto]

LGTM. In terms of format it is very organized, in terms of content nothing stands out to be added or modified at the moment. Anything that needs to be record in JIRA or Confluence for the template?

[jpower432]

LGTM. In terms of format it is very organized, in terms of content nothing stands out to be added or modified at the moment. Anything that needs to be record in JIRA or Confluence for the template?

Thanks @beatrizmcouto. I don't think we need to record anything separately. This should just apply to this repository.

[beatrizmcouto]

@afflom before you joined the daily Jenn and I were discussing on creating a checklist of things to do for the person who is reviewing a PR and ensure we all agree on the approach. Could you review by the end of the week so that we can discuss any ideas as a team next week? I will add it to my to do list as well.