Skip to content

Commit 6538f37

Browse files
GarboMuffinDNin01
andauthored
Apply suggestion from @DNin01
Co-authored-by: DNin01 <106490990+DNin01@users.noreply.github.com>
1 parent 85af43f commit 6538f37

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

REVIEW_PROCESS.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ The requirements for merging a pull request are:
99

1010
**That's it. If everything is green, go ahead and merge it.** The rest of this document will try to define what an approval means.
1111

12-
I'm not expecting you to review things to the same standard I tried to. That would be hopeless. Just make a good effort. You will miss things. That's fine. I missed a lot too.
12+
I'm not expecting you to review things to the same standard I tried to. That would be hopeless. Just do your best and make a good effort. If you miss things, that's fine. I missed a lot too.
1313

1414
Having to copy and paste the templates might seem a bit silly, but this way it's clear to everyone what the expectations are (no one reads CONTRIBUTING.md but people do read review comments). You can add anything else before or after. If reviewers keep not using the templates, we'll have a bot automatically discard approvals that don't use the template. If you just use the templates then we won't need to bother writing that.
1515

0 commit comments

Comments
 (0)