Commit issues

.github/PULL_REQUEST_TEMPLATE.md

@@ -17,10 +17,12 @@ about: A template PR for code review with a checklist
 ### Files
 
 - [ ] The file name describes the function's behavior
+- [ ] There is a module header in the function file
 - [ ] There is a module docstring in the function file
 - [ ] The test file's name matches the function file name -
   `/tests/test_file_name.py`
 - [ ] There is a module docstring in the tests file
+- [ ] There is a module header in the tests file
 
 ### Unit Tests
 
@@ -54,7 +56,10 @@ 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, 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_
 
 ## Strategy
 
@@ -66,7 +71,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
@@ -82,5 +87,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`  or `input` statements anywhere in the function or test files
 - [ ] The code includes defensive assertions
 - [ ] Defensive assertions include as little logic as possible

.vscode/settings.json

@@ -119,8 +119,8 @@
     "editor.defaultFormatter": "charliermarsh.ruff",
     "editor.formatOnSave": true,
     "editor.codeActionsOnSave": {
-      "source.fixAll.ruff": true,
-      "source.organizeImports.ruff": true
+      "source.fixAll.ruff": "explicit",
+      "source.organizeImports.ruff": "explicit"
     }
   }
 }