Skip to content

Commit 85eff5d

Browse files
authored
Merge pull request #2590 from anothertobi/master
Adds the 'trivial edit contribution policy' to contributors/guide/pull-requests.md
2 parents 96ea66b + 15c88b5 commit 85eff5d

File tree

1 file changed

+15
-0
lines changed

1 file changed

+15
-0
lines changed

contributors/guide/pull-requests.md

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ This doc explains the process and best practices for submitting a pull request t
2323
* [7. KISS, YAGNI, MVP, etc.](#7-kiss-yagni-mvp-etc)
2424
* [8. It's OK to Push Back](#8-its-ok-to-push-back)
2525
* [9. Common Sense and Courtesy](#9-common-sense-and-courtesy)
26+
* [10. Trivial Edits](#10-trivial-edits)
2627

2728
# Before You Submit a Pull Request
2829

@@ -283,3 +284,17 @@ Another phenomenon of open-source projects (where anyone can comment on any issu
283284

284285
No document can take the place of common sense and good taste. Use your best judgment, while you put
285286
a bit of thought into how your work can be made easier to review. If you do these things your pull requests will get merged with less friction.
287+
288+
## 10. Trivial Edits
289+
290+
Each incoming Pull Request needs to be reviewed, checked, and then merged.
291+
292+
While automation helps with this, each contribution also has an engineering cost. Therefore it is appreciated if you do NOT make trivial edits and fixes, but instead focus on giving the entire file a review.
293+
294+
If you find one grammatical or spelling error, it is likely there are more in that file, you can really make your Pull Request count by checking formatting, checking for broken links, and fixing errors and then submitting all the fixes at once to that file.
295+
296+
**Some questions to consider:**
297+
298+
* Can the file be improved further?
299+
* Does the trivial edit greatly improve the quality of the content?
300+

0 commit comments

Comments
 (0)