Skip to content

Commit

Permalink
Merge branch '8-is-number-odd' of https://github.com/MIT-Emerging-Tal…
Browse files Browse the repository at this point in the history
…ent/ET6-foundations-group-25 into 8-is-number-odd
  • Loading branch information
azziimm7 committed Jan 8, 2025
2 parents ac940b2 + 3596afd commit 740e74b
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,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
- _It's ok to have extra helper functions if necessary, like with mergesort_
- [ ] The function has correct type annotations
- [ ] The function is not called at the top level of the function file
- _Recursive solutions **can** call the function from **inside** the function body_
Expand All @@ -68,7 +69,7 @@ about: A template PR for code review with a checklist

### Don'ts

- [ ] The function's strategy _is not_ described in the documentation
- [ ] The function's strategy _is not_ described in any docstrings or tests
- [ ] Comments explain the _strategy_, **not** the _implementation_
- [ ] The function _does not_ have more comments than code
- If it does, consider finding a new strategy or a simpler implementation
Expand Down

0 comments on commit 740e74b

Please sign in to comment.