Skip to content

Commit c1b8d23

Browse files
jotakmemodi
andauthored
update PR template (#2515)
* update PR template * Apply suggestion from @memodi Co-authored-by: Mehul Modi <memodi@redhat.com> * Update .github/pull_request_template.md Co-authored-by: Mehul Modi <memodi@redhat.com> --------- Co-authored-by: Mehul Modi <memodi@redhat.com>
1 parent 650a60c commit c1b8d23

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

.github/pull_request_template.md

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -9,15 +9,11 @@ n/a
99

1010
## Checklist
1111

12-
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.
12+
<!-- If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that. -->
1313

14-
* [ ] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix _(in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes)._
15-
* [ ] Does this PR require product documentation?
16-
* [ ] If so, make sure the JIRA epic is labeled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.
17-
* [ ] Does this PR require a product release notes entry?
18-
* [ ] If so, fill in "Release Note Text" in the JIRA.
19-
* [ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.
20-
* [ ] If so, make sure it is described in the JIRA ticket.
14+
* [ ] Does the changes in PR need specific configuration or environment set up for testing?
15+
* [ ] if so please describe it in PR description.
16+
* [ ] I have added thorough unit tests for the change.
2117
* QE requirements (check 1 from the list):
2218
* [ ] Standard QE validation, with pre-merge tests unless stated otherwise.
2319
* [ ] Regression tests only (e.g. refactoring with no user-facing change).

0 commit comments

Comments
 (0)