18 lines
863 B
Markdown
18 lines
863 B
Markdown
|
|
||
|
|
|
||
|
|
|
||
|
|
## Code Review Checklist
|
||
|
|
- [ ] Does the PR describe what changes are being made?
|
||
|
|
- [ ] Does the PR describe why the changes are being made?
|
||
|
|
- [ ] Does the code follow [our style guide](https://github.com/storj/docs/blob/master/code/Style.md)?
|
||
|
|
- [ ] Does the code follow [our testing guide](https://github.com/storj/docs/blob/master/code/Testing.md)?
|
||
|
|
- [ ] Could the PR be broken into smaller PRs?
|
||
|
|
- [ ] Does the new code have enough tests? (*every* PR should have tests or justification otherwise. Bug-fix PRs especially)
|
||
|
|
- [ ] Does the new code have enough documentation that answers "how do I use it?" and "what does it do?"? (both source documentation and [higher level](https://github.com/storj/docs), diagrams?)
|
||
|
|
- [ ] Does any documentation need updating?
|
||
|
|
|
||
|
|
|
||
|
|
Thanks for submitting a PR!
|
||
|
|
|
||
|
|
|