PULL_REQUEST_TEMPLATE.md

What does this PR do?

Required

How can this change be manually tested?

Required

Asana Card

Required

Questions or concerns about this change?

Optional

Deployment Notes

Optional, e.g. caching, data migration, schema changes, etc

---

PR Readiness

• Changes have been validated in local dev

Code Readiness

• Code follows AT Media styles and conventions
• Code has been auto-formatted and is free of lint errors
• Unrelated code changes are not present
• Relevant inline documentation or READMEs have been updated
• Application and browser logs reviewed for runtime warnings and errors
• Correct level of logging added to code

Automated Testing

• Unit tests are written at the appropriate level given the code’s risk-level
• Existing tests still pass
• No unnecessary test runner output, e.g. debugging or warnings are not present in Jest or Unittest output

Other Considerations

• Relevant documentation updated in Slite

Also consider this PR's impact on architecture and security; AMP, Apple News, RSS; SEO, GA, performance and accessibility.

Please see Post Merge for additional responsibilities after this PR is merged.