Closed abhsheksingh closed 9 months ago
We wanted the workflow to build and deploy MkDocs pages upon merging a pull request into the master branch.
Fixes #
Any new code includes all the following forms of documentation:
parameters
returns
requirements.txt
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.
We wanted the workflow to build and deploy MkDocs pages upon merging a pull request into the master branch.
Pull Request submission
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.