blob: eea74002522776f66bc24a8e30842c8fd1d436ff [file] [log] [blame]
Jason M. Bills0e5af6e2020-04-08 13:44:38 -07001List of maintainers for service-config-manager
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
20 X: NAME <EMAIL_USERNAME@DOMAIN> <IRC_USERNAME!>
21 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
30 M: Denotes maintainer; has fields NAME <EMAIL_USERNAME@DOMAIN> <IRC_USERNAME!>;
31 if omitted from an entry, assume one of the maintainers from the
32 MAINTAINERS entry.
33 R: Denotes reviewer; has fields NAME <EMAIL_USERNAME@DOMAIN> <IRC_USERNAME!>;
34 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.
53 - Feel free to ping on IRC about patches that look good but have not
54 received +2
55
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: AppaRao Puli <apparao.puli@linux.intel.com>
66M: Richard Marian Thomaiyar <richard.marian.thomaiyar@linux.intel.com>