Skip to content

Commit 6bfdc92

Browse files
benglMylesBorins
authored andcommitted
doc: clarify "Reviewed-By" iff "LGTM"
As per conversation with @Trott, make it clear that Reviewed-By lines should only be added for collaborators who've actually put a LGTM on the PR. PR-URL: #7183 Reviewed-By: Myles Borins <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: JacksonTian - Jackson Tian <[email protected]>
1 parent afbfbc0 commit 6bfdc92

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

doc/onboarding.md

+1
Original file line numberDiff line numberDiff line change
@@ -165,6 +165,7 @@ Landing a PR
165165
* `Reviewed-By: human <email>`
166166
* Easiest to use `git log` then do a search
167167
* (`/Name` + `enter` (+ `n` as much as you need to) in vim)
168+
* Only include collaborators who have commented "LGTM"
168169
* `PR-URL: <full-pr-url>`
169170
* `git push upstream master`
170171
* close the original PR with "Landed in `<commit hash>`".

0 commit comments

Comments
 (0)