@cryptotaxi247 / netdata-1 / commits / 9faa03df5

Improve Pull Request template to have a shorter testing section with enhanced instructions (#8357)

* Improve Pull Request tempalte to have a shorter testing section with enhanced instructions * Update .github/PULL_REQUEST_TEMPLATE.md Co-Authored-By: Patti Short <35278231+shortpatti@users.noreply.github.com> * Update .github/PULL_REQUEST_TEMPLATE.md Co-Authored-By: Patti Short <35278231+shortpatti@users.noreply.github.com> * Update .github/PULL_REQUEST_TEMPLATE.md Co-Authored-By: Patti Short <35278231+shortpatti@users.noreply.github.com> * Update .github/PULL_REQUEST_TEMPLATE.md Co-Authored-By: Patti Short <35278231+shortpatti@users.noreply.github.com> * Update .github/PULL_REQUEST_TEMPLATE.md Co-Authored-By: Patti Short <35278231+shortpatti@users.noreply.github.com> Co-authored-by: Patti Short <35278231+shortpatti@users.noreply.github.com>

James Mills committed Mar 12, 2020 at 08:02 UTC 9faa03df57682204290db12ea2bda032b2d16807
1 file changed +7 -5
.github/PULL_REQUEST_TEMPLATE.md
+7 -5
@@ -5,6 +5,8 @@ Include "Fixes #nnn" if you are fixing an existing issue.
5 In "Component Name" section write which component is changed in this PR. This
6 will help us review your PR quicker.
7
8 +In "Test Plan" provide enough detail on how you plan to test this PR so that a reviewer can validate your tests. If our CI covers sufficient tests, then state which tests cover the change.
9 +
10 If you have more information you want to add, write them in "Additional
11 Information" section. This is usually used to help others understand your
12 motivation behind this change. A step-by-step reproduction of the problem is
@@ -15,12 +17,12 @@ helpful if there is no related issue.
17
18 ##### Component Name
19
18 -##### Description of testing that the developer performed
20 +##### Test Plan
21
22 <!---
21 -Please be detailed enough that your reviewer can understand which test-cases you
22 -have covered, and recreate them if necessary.
23 +Provide enough detail so that your reviewer can understand which test-cases you
24 +have covered, and recreate them if necessary. If sufficient tests are covered
25 +by our CI, then state which tests cover the change.
26 -->
24 -##### Additional Information
25 -
27
28 +##### Additional Information