Closed james-westwood closed 1 year ago
Order to merge pull requests: 1) #360 2) This one: #361 3) #366
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.
Order to merge pull requests: 1) #360 2) This one: #361 3) #366
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.