diff options
authorEmily Shaffer <>2018-04-02 17:03:31 -0700
committerVernon Mauery <>2018-05-01 15:29:11 +0000
commit1979145da515d0ae258892e357d103334a484aad (patch)
parent187f564a35235d38a59af3dfb18bb76883851d30 (diff)
MAINTAINERS: add repo-local MAINTAINERS file
Change-Id: Id93e51694e527790209ad4511e4cefb154fe9887 Signed-off-by: Emily Shaffer <>
1 files changed, 69 insertions, 0 deletions
new file mode 100644
index 0000000..4aa823b
--- /dev/null
@@ -0,0 +1,69 @@
+List of maintainers for phosphor-host-ipmid
+How to use this list:
+ Find the most specific section entry (described below) that matches where
+ your change lives and add the reviewers (R) and maintainers (M) as
+ reviewers. You can use the same method to track down who knows a particular
+ code base best.
+ Your change/query may span multiple entries; that is okay.
+ If you do not find an entry that describes your request at all, someone
+ forgot to update this list; please at least file an issue or send an email
+ to a maintainer, but preferably you should just update this document.
+Description of section entries:
+ Section entries are structured according to the following scheme:
+ X: ...
+ .
+ .
+ .
+ Where REPO_NAME is the name of the repository within the OpenBMC GitHub
+ organization; FILE_PATH is a file path within the repository, possibly with
+ wildcards; X is a tag of one of the following types:
+ M: Denotes maintainer; has fields NAME <EMAIL_USERNAME@DOMAIN> <IRC_USERNAME!>;
+ if omitted from an entry, assume one of the maintainers from the
+ R: Denotes reviewer; has fields NAME <EMAIL_USERNAME@DOMAIN> <IRC_USERNAME!>;
+ these people are to be added as reviewers for a change matching the repo
+ path.
+ F: Denotes forked from an external repository; has fields URL.
+ Line comments are to be denoted "# SOME COMMENT" (typical shell style
+ comment); it is important to follow the correct syntax and semantics as we
+ may want to use automated tools with this file in the future.
+ A change cannot be added to an OpenBMC repository without a MAINTAINER's
+ approval; thus, a MAINTAINER should always be listed as a reviewer.
+Change approval rules:
+ - Patches must be available for review for a minimum of 48 hours before it
+ can be submitted.
+ - Patches must be be approved (+1) by at least 2 maintainers.
+ - Patches must not have an unresolved -1 vote by any maintainer.
+ - Patches should have all maintainers added for visibility.
+ - Patches should include unit tests where possible.
+ - Feel free to ping on IRC about patches that look good but have not
+ received +2
+Design approval rules:
+ - Design discussions should be carried out via email with, at minimum,
+ all maintainers on the thread. It's encouraged to include the
+ OpenBMC mailing list in the thread as well.
+M: Vernon Mauery <> <vmauery!>
+M: Tom Joseph <> <tomjoseph!>
+M: Emily Shaffer <> <emilyshaffer!>
+R: Adriana Kobylak <> <anoo!>
+R: Deepak Kodihalli <> <dkodihal!>
OpenPOWER on IntegriCloud