diff options
author | Emily Shaffer <emilyshaffer@google.com> | 2018-04-02 17:03:31 -0700 |
---|---|---|
committer | Vernon Mauery <vernon.mauery@linux.intel.com> | 2018-05-01 15:29:11 +0000 |
commit | 1979145da515d0ae258892e357d103334a484aad (patch) | |
tree | 1283997627b2399a01e3b20eb1350031b564c264 | |
parent | 187f564a35235d38a59af3dfb18bb76883851d30 (diff) | |
download | phosphor-host-ipmid-1979145da515d0ae258892e357d103334a484aad.tar.gz phosphor-host-ipmid-1979145da515d0ae258892e357d103334a484aad.zip |
MAINTAINERS: add repo-local MAINTAINERS file
Change-Id: Id93e51694e527790209ad4511e4cefb154fe9887
Signed-off-by: Emily Shaffer <emilyshaffer@google.com>
-rw-r--r-- | MAINTAINERS | 69 |
1 files changed, 69 insertions, 0 deletions
diff --git a/MAINTAINERS b/MAINTAINERS new file mode 100644 index 0000000..4aa823b --- /dev/null +++ b/MAINTAINERS @@ -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: NAME <EMAIL_USERNAME@DOMAIN> <IRC_USERNAME!> + 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 + MAINTAINERS entry. + 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. + +START OF MAINTAINERS LIST +------------------------- + +M: Vernon Mauery <vernon.mauery@linux.intel.com> <vmauery!> +M: Tom Joseph <tomjose@linux.vnet.ibm.com> <tomjoseph!> +M: Emily Shaffer <emilyshaffer@google.com> <emilyshaffer!> +R: Adriana Kobylak <anoo@us.ibm.com> <anoo!> +R: Deepak Kodihalli <dkodihal@linux.vnet.ibm.com> <dkodihal!> |