Skip to content

Commit

Permalink
Merge pull request #38 from MIT-Emerging-Talent/main
Browse files Browse the repository at this point in the history
Update fork with upstream repository
  • Loading branch information
AnaiMurillo authored Jan 4, 2025
2 parents a14d982 + 363e84b commit 442f1d7
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ about: A template PR for code review with a checklist
- [ ] The function's name describes it's behavior
- [ ] The function's name matches the file name
- [ ] The function has correct type annotations
- [ ] The function is not called in the function file

## Strategy

Expand Down Expand Up @@ -82,5 +83,7 @@ about: A template PR for code review with a checklist
- [ ] Variable names are clear and helpful
- [ ] The code follows the strategy as simply as possible
- [ ] The implementation is as simple as possible given the strategy
- [ ] There are no commented lines of code
- [ ] There are no `print` statements anywhere
- [ ] The code includes defensive assertions
- [ ] Defensive assertions include as little logic as possible

0 comments on commit 442f1d7

Please sign in to comment.