Commit 54fa32f
committed
MAINTAINERS_GUIDE: Mention email in "How are decisions made?"
The old wording did not mention email discussion before working up
changes, which we often recommend [1,2] to avoid contributors sinking
a lot of work into a pull request that ends up being rejected because
of a fundamental design issue. The new wording mentions that and
also:
* Removes the overly compact short answer to avoid confusion [3]. The
section is not so long that it needs a one-line summary.
* Distinguishes between in-PR votes (LGTM/Rejected) and
merging/closing the PR.
* Mentions GOVERNANCE for management changes.
* Uses an enumerated list instead of "Step N" text.
* Uses the README's recommended one line per sentence.
[1]: CONTRIBUTING.md#conventions
[2]: opencontainers/runtime-spec#420 (comment)
[3]: opencontainers/runtime-spec#420 (comment)
Signed-off-by: W. Trevor King <[email protected]>1 parent 62bcb78 commit 54fa32f
1 file changed
+13
-10
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
33 | | - | |
34 | | - | |
35 | 33 | | |
36 | 34 | | |
37 | 35 | | |
| |||
43 | 41 | | |
44 | 42 | | |
45 | 43 | | |
46 | | - | |
47 | | - | |
48 | | - | |
49 | | - | |
50 | | - | |
51 | | - | |
52 | | - | |
53 | | - | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
54 | 57 | | |
55 | 58 | | |
56 | 59 | | |
| |||
0 commit comments