blob: a9e4987822fe1958f301aa4a5c06a0d3e69b8830 [file] [log] [blame]
Emily Shaffer19791452018-04-02 17:03:31 -07001List of maintainers for phosphor-host-ipmid
2===============================
3
4How to use this list:
5 Find the most specific section entry (described below) that matches where
6 your change lives and add the reviewers (R) and maintainers (M) as
7 reviewers. You can use the same method to track down who knows a particular
8 code base best.
9
10 Your change/query may span multiple entries; that is okay.
11
12 If you do not find an entry that describes your request at all, someone
13 forgot to update this list; please at least file an issue or send an email
14 to a maintainer, but preferably you should just update this document.
15
16Description of section entries:
17
18 Section entries are structured according to the following scheme:
19
Bruce Mitchell66ae67d2021-04-12 13:00:32 -050020 X: NAME <EMAIL_USERNAME@DOMAIN> <DISCORD_USERNAME!>
Emily Shaffer19791452018-04-02 17:03:31 -070021 X: ...
22 .
23 .
24 .
25
26 Where REPO_NAME is the name of the repository within the OpenBMC GitHub
27 organization; FILE_PATH is a file path within the repository, possibly with
28 wildcards; X is a tag of one of the following types:
29
Bruce Mitchell66ae67d2021-04-12 13:00:32 -050030 M: Denotes maintainer; has fields NAME <EMAIL_USERNAME@DOMAIN> <DISCORD_USERNAME!>;
Emily Shaffer19791452018-04-02 17:03:31 -070031 if omitted from an entry, assume one of the maintainers from the
32 MAINTAINERS entry.
Bruce Mitchell66ae67d2021-04-12 13:00:32 -050033 R: Denotes reviewer; has fields NAME <EMAIL_USERNAME@DOMAIN> <DISCORD_USERNAME!>;
Emily Shaffer19791452018-04-02 17:03:31 -070034 these people are to be added as reviewers for a change matching the repo
35 path.
36 F: Denotes forked from an external repository; has fields URL.
37
38 Line comments are to be denoted "# SOME COMMENT" (typical shell style
39 comment); it is important to follow the correct syntax and semantics as we
40 may want to use automated tools with this file in the future.
41
42 A change cannot be added to an OpenBMC repository without a MAINTAINER's
43 approval; thus, a MAINTAINER should always be listed as a reviewer.
44
45Change approval rules:
46
47 - Patches must be available for review for a minimum of 48 hours before it
48 can be submitted.
49 - Patches must be be approved (+1) by at least 2 maintainers.
50 - Patches must not have an unresolved -1 vote by any maintainer.
51 - Patches should have all maintainers added for visibility.
52 - Patches should include unit tests where possible.
Bruce Mitchellc874be82021-04-08 16:48:34 -050053 - Feel free to ping on Discord about patches that look good but have
54 not received +2
Emily Shaffer19791452018-04-02 17:03:31 -070055
56Design approval rules:
57
58 - Design discussions should be carried out via email with, at minimum,
59 all maintainers on the thread. It's encouraged to include the
60 OpenBMC mailing list in the thread as well.
61
62START OF MAINTAINERS LIST
63-------------------------
64
65M: Vernon Mauery <vernon.mauery@linux.intel.com> <vmauery!>
Tom Joseph7cb35502021-06-09 04:51:31 -070066M: Tom Joseph <rushtotom@gmail.com> <tomjose!>
Emily Shaffer19791452018-04-02 17:03:31 -070067R: Adriana Kobylak <anoo@us.ibm.com> <anoo!>
Deepak Kodihallic0fe0c62020-12-07 15:33:31 +053068R: Deepak Kodihalli <deepak.kodihalli.83@gmail.com> <dkodihal!>
Ratan Gupta1be10c32018-05-21 12:09:08 +053069R: Ratan Gupta <ratagupt@linux.vnet.ibm.com> <rgupta!>