How does it accomplish it, in broad strokes? i.e. How does it change the codebase?
Interesting/controversial decisions
Include anything that you thought twice about, debated, chose arbitrarily, etc.
What could have been done differently, but wasn't? And why?
Test coverage
Have you included tests (which could be a transcript) for this change, or is it somehow covered by existing tests?
Would you recommend improving the test coverage (either as part of this MR or as a separate issue) or do you think it’s adequate?
Loose ends
Link to related issues that address things you didn't get to. Stuff you encountered on the way and decided not to include in this MR.Give a brief description of the changes made
Title of changes made (change this line)
Overview
WIP
Tickets
List the tickets of the PR linked
Implementation notes
How does it accomplish it, in broad strokes? i.e. How does it change the codebase?
Interesting/controversial decisions
Include anything that you thought twice about, debated, chose arbitrarily, etc. What could have been done differently, but wasn't? And why?
Test coverage
Have you included tests (which could be a transcript) for this change, or is it somehow covered by existing tests?
Would you recommend improving the test coverage (either as part of this MR or as a separate issue) or do you think it’s adequate?
Loose ends
Link to related issues that address things you didn't get to. Stuff you encountered on the way and decided not to include in this MR.Give a brief description of the changes made