Skip to content

Commit

Permalink
Remove section about self-approve patches
Browse files Browse the repository at this point in the history
We moved away from that model and every commit should go
through a review.
  • Loading branch information
mrunge committed Feb 19, 2020
1 parent 47c4fa6 commit 9b71b27
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions docs/maintainer_guide.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,14 +24,3 @@ using your commit rights to merge the pull request.

You have write access to the *collectd/collectd* repository. Please use it
responsibly.

#### Own work

Open *pull requests* for your own changes, too:

* For simple changes it's okay to self-approve and merge after a
successful build on the CI systems.
* Trivial changes, cherry-picks from *master* and roll-up merges are
excempt and may be pushed to the version branches and *master* directly.
* "Simple" and "trivial" are not further defined; use your best judgement.
We'll revisit this if and when it becomes necessary.

0 comments on commit 9b71b27

Please sign in to comment.