pr template: migrations run concurrently with api servers now
This commit is contained in:
parent
43103ae13f
commit
8601f54f32
2
.github/pull_request_template.md
vendored
2
.github/pull_request_template.md
vendored
@ -10,6 +10,7 @@ Please describe the tests:
|
||||
Please describe the performance impact:
|
||||
|
||||
## Code Review Checklist (to be filled out by reviewer)
|
||||
- [ ] NEW: Are there any Satellite database migrations? Are they forwards _and_ backwards compatible?
|
||||
- [ ] 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)?
|
||||
@ -19,3 +20,4 @@ Please describe the performance impact:
|
||||
- [ ] 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?
|
||||
- [ ] Do the database access patterns make sense?
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user