Skip to content

Commit

Permalink
Merge pull request opencontainers#13 from cyphar/disallow-self-lgtm
Browse files Browse the repository at this point in the history
MAINTAINERS: disallow self-LGTMs
  • Loading branch information
caniszczyk committed May 31, 2016
2 parents 560fdc9 + c82a2e7 commit 1d5bddc
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion MAINTAINERS_GUIDE.md
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,9 @@ All decisions are pull requests, and the relevant maintainers make
decisions by accepting or refusing the pull request. Review and acceptance
by anyone is denoted by adding a comment in the pull request: `LGTM`.
However, only currently listed `MAINTAINERS` are counted towards the required
two LGTMs.
two LGTMs. In addition, if a maintainer has created a pull request, they cannot
count toward the two LGTM rule (to ensure equal amounts of review for every pull
request, no matter who wrote it).

Overall the maintainer system works because of mutual respect across the
maintainers of the project. The maintainers trust one another to make decisions
Expand Down

0 comments on commit 1d5bddc

Please sign in to comment.