summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTina Luedtke <kickoke@users.noreply.github.com>2022-03-23 16:56:26 +0000
committerGitHub <noreply@github.com>2022-03-23 16:56:26 +0000
commitc29eae629aa69d3e95e0298a53746d9abbf3facf (patch)
tree50de0346272f62508b4dd989fdd3a0baf777a95a
parent1023c3e196fafcc21c10bf3c3c95b88339b82f1a (diff)
PRs template: Include user information section kickoke-PR-template
##### Summary <!-- Describe the change in summary section, including rationale and design decisions. Include "Fixes #nnn" if you are fixing an existing issue. --> As @thiagoftsm and @Ferroin suggested: We include a list of our PRs in the release notes. However, when you click on a PR as a user, who isn't involved in the day to day business, you might not understand what the PR does. This PR introduces a new section within "Additional information" to provide additional context for our users. ##### Test Plan <!-- Provide enough detail so that your reviewer can understand which test cases you have covered, and recreate them if necessary. If our CI covers sufficient tests, then state which tests cover the change. --> N/A ##### Additional Information <!-- This is usually used to help others understand your motivation behind this change. A step-by-step reproduction of the problem is helpful if there is no related issue. --> The `details` tag renders a dropdown section and the `summary` tag is used to provide a custom heading. Please review this PR as source code and not the rendered version. The `details` tag contains a comment with guiding questions on how to write a good `For users` section. <details> <summary>For users: How does this change affect me?</summary> <!-- Describe the PR affects users: - Which area of Netdata is affected by the change? - Can they see the change or is it an under the hood? If they can see it, where? - How is the user impacted by the change? - What are there any benefits of the change? --> </details>
-rw-r--r--.github/PULL_REQUEST_TEMPLATE.md30
1 files changed, 17 insertions, 13 deletions
diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
index 74f9918e27..829d8e647c 100644
--- a/.github/PULL_REQUEST_TEMPLATE.md
+++ b/.github/PULL_REQUEST_TEMPLATE.md
@@ -1,23 +1,27 @@
+##### Summary
<!--
Describe the change in summary section, including rationale and design decisions.
Include "Fixes #nnn" if you are fixing an existing issue.
-
-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.
-
-If you have more information you want to add, write them in "Additional
-Information" section. This is usually used to help others understand your
-motivation behind this change. A step-by-step reproduction of the problem is
-helpful if there is no related issue.
-->
-##### Summary
-
##### Test Plan
-<!---
-Provide enough detail so that your reviewer can understand which test-cases you
-have covered, and recreate them if necessary. If sufficient tests are covered
-by our CI, then state which tests cover the change.
+<!--
+Provide enough detail so that your reviewer can understand which test cases you
+have covered, and recreate them if necessary. If our CI covers sufficient tests, then state which tests cover the change.
-->
##### Additional Information
+<!-- This is usually used to help others understand your
+motivation behind this change. A step-by-step reproduction of the problem is
+helpful if there is no related issue. -->
+
+<details> <summary>For users: How does this change affect me?</summary>
+ <!--
+Describe the PR affects users:
+- Which area of Netdata is affected by the change?
+- Can they see the change or is it an under the hood? If they can see it, where?
+- How is the user impacted by the change?
+- What are there any benefits of the change?
+-->
+</details>