Please go through this checklist before submitting the PR:
- Is it tested enough (in terms of coverage and depth) by the developer and a third person?
- Were errors handled properly?
- Are deployment steps clearly documented? Is it all automated, or do we need a runbook or something?
- Are possible side effects documented clearly?
- Is there a rollback plan in place, and is there any possibility that the rollback plan may not work as expected, such as uncovered possibilities and unguaranteed actions?
- Are related documents updated? (It may be in multiple places depending on the nature of the change.)
- Are the code and documents clear enough for a newcomer?
- Is there technical debt, and are tech debt tickets already created?