Skip to content
Snippets Groups Projects
Commit 0e6066a4 authored by Colton Leekley-Winslow's avatar Colton Leekley-Winslow
Browse files

touch up pull request guidelines

parent 91e83a54
No related branches found
No related tags found
No related merge requests found
......@@ -17,9 +17,11 @@ Then review and decide what upgrades to make, if any.
Changes to `requirements.txt` should always be by pull request.
## Code Standards
### Testing
- Add tests for every change made
- Tests should be realistic and exercise a wide range of possibilities
### Docstrings
- Use [Google Style Docstrings](http://sphinxcontrib-napoleon.readthedocs.io/en/latest/example_google.html).
- Add docstrings to all functions with a one-line description of its purpose.
......
### Checklist for Reviewer(s)
### Tips for Review
- [ ] Tests were added to cover all code changed
- [ ] Documentation was added / updated
- [ ] Code and tests follow standards in CONTRIBUTING.md
- [ ] A detailed explanation of issues was given by the reviewer, using clear language and technical terms
- Tests were added to cover all code changed
- Documentation was added / updated
- Code and tests follow standards in CONTRIBUTING.md
- A detailed explanation of issues was given by the reviewer, using clear language and technical terms
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment