Skip to content

Commit a42f78b

Browse files
authored
Update CONTRIBUTING.md
Trying to explain why any PR should have a`description`
1 parent 9d70ec5 commit a42f78b

File tree

1 file changed

+11
-1
lines changed

1 file changed

+11
-1
lines changed

CONTRIBUTING.md

+11-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,16 @@
22

33
Here are a few suggestions to make it easier for you to contribute!
44

5+
### Describe your PR
6+
7+
Please add a description of your proposed code changes. It does not need to be an exhaustive essay, however a PR with no description or just a few words might not get accepted, simply because very basic information is missing.
8+
9+
A good description helps us to review and understand your proposed changes. For example, you could say a few words about
10+
* what you try to achieve (new feature, fixing a bug, refactoring, security enhancements, etc.)
11+
* how your code works (short technical summary - focus on important aspects that might not be obvious when reading the code)
12+
* testing you performed, known limitations, open ends you possibly could not solve.
13+
* any areas where you like to get help from an experienced maintainer (yes WLED has become big 😉)
14+
515
### Target branch for pull requests
616

717
Please make all PRs against the `0_15` branch.
@@ -79,4 +89,4 @@ Good:
7989

8090
There is no hard character limit for a comment within a line,
8191
though as a rule of thumb consider wrapping after 120 characters.
82-
Inline comments are OK if they describe that line only and are not exceedingly wide.
92+
Inline comments are OK if they describe that line only and are not exceedingly wide.

0 commit comments

Comments
 (0)