Skip to content

Commit

Permalink
Update PR template
Browse files Browse the repository at this point in the history
The checklist-style for deployment made the pr overview screen really
useless because every PR now had a checklist associated with it.

Instead, let's just delete the servers that don't need deployment from
the list. Same thing for the db migrations, if people need to run one
after merging leave the section in, otherwise delete it.
  • Loading branch information
mxstbr committed Jan 17, 2018
1 parent 6f57f22 commit 505de0f
Showing 1 changed file with 11 additions and 15 deletions.
26 changes: 11 additions & 15 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,18 +1,14 @@
### Deployment notes
-

### Will deploy
- [ ] iris
- [ ] hyperion
- [ ] athena
- [ ] vulcan
- [ ] mercury
- [ ] hermes
- [ ] chronos

### Run database migration?
- [ ] yes
- [ ] no
### Deploy after merge (delete what needn't be deployed)
- iris
- hyperion
- athena
- vulcan
- mercury
- hermes
- chronos

### Run database migrations (delete if not)
YES

## Release notes
-
Expand Down

0 comments on commit 505de0f

Please sign in to comment.