Skip to content

Commit b746e03

Browse files
authored
Merge pull request #286 from YuPengZTE/devTrailing
fix typo and remove trailing blank
2 parents 893d9e3 + dda5403 commit b746e03

1 file changed

Lines changed: 17 additions & 17 deletions

File tree

MAINTAINERS_GUIDE.md

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ features to the project. But most of your time will be spent reviewing,
77
cleaning up, documenting, answering questions, justifying design
88
decisions - while everyone has all the fun! But remember - the quality
99
of the maintainers work is what distinguishes the good projects from the
10-
great. So please be proud of your work, even the unglamourous parts,
10+
great. So please be proud of your work, even the unglamorous parts,
1111
and encourage a culture of appreciation and respect for *every* aspect
1212
of improving the project - not just the hot new features.
1313

@@ -62,43 +62,43 @@ made through a pull request.
6262

6363
All decisions are pull requests, and the relevant maintainers make
6464
decisions by accepting or refusing the pull request. Review and acceptance
65-
by anyone is denoted by adding a comment in the pull request: `LGTM`.
65+
by anyone is denoted by adding a comment in the pull request: `LGTM`.
6666
However, only currently listed `MAINTAINERS` are counted towards the required
6767
two LGTMs.
6868

6969
Overall the maintainer system works because of mutual respect across the
7070
maintainers of the project. The maintainers trust one another to make decisions
71-
in the best interests of the project. Sometimes maintainers can disagree and
71+
in the best interests of the project. Sometimes maintainers can disagree and
7272
this is part of a healthy project to represent the point of views of various people.
73-
In the case where maintainers cannot find agreement on a specific change the
74-
role of a Chief Maintainer comes into play.
73+
In the case where maintainers cannot find agreement on a specific change the
74+
role of a Chief Maintainer comes into play.
7575

76-
The Chief Maintainer for the project is responsible for overall architecture
77-
of the project to maintain conceptual integrity. Large decisions and
78-
architecture changes should be reviewed by the chief maintainer.
79-
The current chief maintainer for the project is the first person listed
80-
in the MAINTAINERS file.
76+
The Chief Maintainer for the project is responsible for overall architecture
77+
of the project to maintain conceptual integrity. Large decisions and
78+
architecture changes should be reviewed by the chief maintainer.
79+
The current chief maintainer for the project is the first person listed
80+
in the MAINTAINERS file.
8181

8282
Even though the maintainer system is built on trust, if there is a conflict
83-
with the chief maintainer on a decision, their decision can be challenged
84-
and brought to the technical oversight board if two-thirds of the
85-
maintainers vote for an appeal. It is expected that this would be a
83+
with the chief maintainer on a decision, their decision can be challenged
84+
and brought to the technical oversight board if two-thirds of the
85+
maintainers vote for an appeal. It is expected that this would be a
8686
very exceptional event.
8787

8888

8989
### How are maintainers added?
9090

9191
The best maintainers have a vested interest in the project. Maintainers
9292
are first and foremost contributors that have shown they are committed to
93-
the long term success of the project. Contributors wanting to become
94-
maintainers are expected to be deeply involved in contributing code,
93+
the long term success of the project. Contributors wanting to become
94+
maintainers are expected to be deeply involved in contributing code,
9595
pull request review, and triage of issues in the project for more than two months.
9696

97-
Just contributing does not make you a maintainer, it is about building trust
97+
Just contributing does not make you a maintainer, it is about building trust
9898
with the current maintainers of the project and being a person that they can
9999
depend on and trust to make decisions in the best interest of the project. The
100100
final vote to add a new maintainer should be approved by over 66% of the current
101-
maintainers with the chief maintainer having veto power. In case of a veto,
101+
maintainers with the chief maintainer having veto power. In case of a veto,
102102
conflict resolution rules expressed above apply. The voting period is
103103
five business days on the Pull Request to add the new maintainer.
104104

0 commit comments

Comments
 (0)