Closed jwestw closed 1 year ago
Insert detailed bullet points about your changes here!
Insert any instructions to help the reviewer, e.g. "install new requirements from requirements.txt"
requirements.txt
*Let the reviewer know what data files are needed (to be grabbed from sync)
Fixes #
Any new code includes all the following forms of documentation:
parameters
returns
The author has responded to my review and made changes to my satisfaction.
Insert detailed comments here!
These might include, but not exclusively:
Your suggestions should be tailored to the code that you are reviewing. Be critical and clear, but not mean. Ask questions and set actions.
Pull Request submission
Insert detailed bullet points about your changes here!
Insert any instructions to help the reviewer, e.g. "install new requirements from
requirements.txt
"*Let the reviewer know what data files are needed (to be grabbed from sync)
Closes or fixes
Fixes #
Code
Documentation
Any new code includes all the following forms of documentation:
parameters
andreturns
for all major functionsData
Testing
Peer Review Section
requirements.txt
Final approval (post-review)
The author has responded to my review and made changes to my satisfaction.
Review comments
Insert detailed comments here!
These might include, but not exclusively:
Your suggestions should be tailored to the code that you are reviewing. Be critical and clear, but not mean. Ask questions and set actions.