Skip to content

Commit

Permalink
Merge pull request #272 from bancorprotocol/pr-template
Browse files Browse the repository at this point in the history
Create pull_request_template.md
  • Loading branch information
zavelevsky authored Dec 25, 2023
2 parents c41f9c5 + ea578ab commit af52961
Showing 1 changed file with 15 additions and 0 deletions.
15 changes: 15 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
## Related Issue
- Link to the related Issue, in the format of `fixes: #<issue_number>`

## Changes Made
- Summary of changes made

## Impact Analysis (here you stop and think)
- Automated Tests: Details on test changes. It should be cleared that this PR doesn't degrade our test coverage and that the changes in this PR are covered by the tests.
- Risks: Rate the risk (High/Medium/Low) and areas to focus in code review and manual tests
- Performance: Measures taken to assess impact
- Monitoring and Logging: Changes made / required and their rationale
- Documentation: List of documentation updated/created

## Additional Notes
- Any other relevant information, including alternatives that were discussed, compromises and scaledowns, etc.

0 comments on commit af52961

Please sign in to comment.