-
Notifications
You must be signed in to change notification settings - Fork 0
Madlibs feedback #1
Copy link
Copy link
Open
Description
natepill
opened on Sep 9, 2019
Issue body actions
- Good variable names
- Good modularity
- Good README
- Try to avoid globally scoped variables, if any exist, put at top of file
- Comments should reflect the “why” behind code, not what it's doing exactly
- Would like to see more comments
- Would like to see docstrings for functions
- Would like to see python main function
- Could benefit from more test cases
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels