Skip to content

Commit 33405a7

Browse files
author
Matthew Fisher
committed
docs(github): simplify github templates
We've found that half of the engineering team either hates or doesn't even bother using the PR templates. Refactoring it down to the bare minimum should suffice.
1 parent 80d8cf9 commit 33405a7

2 files changed

Lines changed: 5 additions & 82 deletions

File tree

.github/ISSUE_TEMPLATE

Lines changed: 0 additions & 36 deletions
This file was deleted.

.github/PULL_REQUEST_TEMPLATE

Lines changed: 5 additions & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -1,49 +1,8 @@
1-
# Summary of Changes
1+
If your change requires any additions or changes to the [documentation][docs] or to the [end-to-end test suite][e2e], please submit them as 1 or more pull requests against that repo and refer to them here.
22

3-
Please explain what this PR changes, and include what unit tests you've written (we hope you wrote tests!)
3+
refs deis/workflow#1234
4+
refs deis/workflow-e2e#5678
45

5-
# Issue(s) that this PR Closes
66

7-
Please list the issue(s) that this PR closes, similar to the below:
8-
9-
Closes #1234
10-
Closes #5678
11-
12-
# Associated [End To End Test](https://github.com/deis/workflow-e2e) PR(s)
13-
14-
If your change requires any additions or changes to the [end to end tests](https://github.com/deis/workflow-e2e), please submit them as 1 or more pull requests against that repo and refer to them here.
15-
16-
Ref deis/workflow-e2e#1234
17-
Ref deis/workflow-e2e#5678
18-
19-
# Associated [Documentation](https://github.com/deis/workflow) PR(s)
20-
21-
If your change requires any additions or changes to the [official documentation](https://github.com/deis/workflow), please submit them as 1 or more pull requests against that repo and refer to them here.
22-
23-
Ref deis/workflow#1234
24-
Ref deis/workflow#5678
25-
26-
# Associated [Design Document](http://docs.deis.io/en/latest/contributing/design-documents)(s)
27-
28-
If your change was inspired by 1 or more design document(s), please provide a list of links to them here.
29-
30-
# Testing Instructions
31-
32-
Please provide a detailed list for how to test the changes in this PR.
33-
34-
1. Create a Deis Cluster
35-
2. Register an app
36-
3. Your Steps Here...
37-
38-
Also, please provide a description of the desired result after the tester completes the above steps.
39-
40-
1. The app called "abcd" should be deployed
41-
2. `kubectl get pod --namespace=abcd` should show 2 pods running
42-
3. etc...
43-
44-
# Pull Request Hygiene TODOs
45-
46-
Please make sure the below checklist is complete.
47-
48-
- [ ] Your pull request is concise and to the point (make another PR for refactoring nearby code)
49-
- [ ] Your commits follow the [commit style guidelines](http://docs.deis.io/en/latest/contributing/standards/#commit-style-guide)
7+
[docs]: https://github.com/deis/workflow
8+
[e2e]: https://github.com/deis/workflow-e2e

0 commit comments

Comments
 (0)