forked from codice/ddf
-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathPULL_REQUEST_TEMPLATE
38 lines (38 loc) · 1.36 KB
/
PULL_REQUEST_TEMPLATE
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
#### What does this PR do?
#### Who is reviewing it?
(please choose AT LEAST two reviewers that need to approve the PR before it can get merged)
#### Select relevant component teams:
https://github.com/orgs/codice/teams
#### Choose 2 committers to review/merge the PR.
(please choose ONLY two committers from below, delete the rest)
@andrewkfiedler
@bdeining
@beyelerb
@clockard
@coyotesqrl
@figliold
@jaymcnallie
@jlcsmith
@kcwire
@lessarderic
@michaelmenousek
@millerw8
@mrmateo
@pklinef
@ricklarsen - Documentation
@rzwiefel
@shaundmorris
@stustison
#### How should this be tested? (List steps with links to updated documentation)
#### Any background context you want to provide?
#### What are the relevant tickets?
[](https://codice.atlassian.net/browse/)
#### Screenshots (if appropriate)
#### Checklist:
- [ ] Documentation Updated
- [ ] Update / Add Unit Tests
- [ ] Update / Add Integration Tests
#### Review Comment Legend:
- ✏️ (Pencil) This comment is a nitpick or style suggestion, no action required for approval. This comment should provide a suggestion either as an in line code snippet or a gist.
- ❓ (Question Mark) This comment is to gain a clearer understanding of design or code choices, clarification is required but action may not be necessary for approval.
- ❗ (Exclamation Mark) This comment is critical and requires clarification or action before approval.