Update page 'Move to Approve'
parent
66bfc98032
commit
cb7f2bd881
|
@ -1 +1,63 @@
|
|||
Welcome to the Wiki.
|
||||
# Maintainer Guidelines
|
||||
The purpose of this document is to manage expectations and make contributing pleasurable.
|
||||
|
||||
We like contributors but, looking at the numbers, most contributors to a project only ever make one commit.
|
||||
They are often one-offs. It is the 1% who move the bulk of the code.
|
||||
|
||||
As such, we want to keep the barrier to entry for contributors as low as possible.
|
||||
We don't see it as reasonable for newcomers to the community to have to become intimately
|
||||
familiar with our way of doing things in order to make a meaningful contribution.
|
||||
|
||||
As such, we believe the responsibility falls to the maintainers to know and effectively communicate
|
||||
the rules of the road and, most importantly: to move to approve, quickly:
|
||||
|
||||
## 1. Move to Approve
|
||||
|
||||
We acknowledge that the reason someone creates a PR is likely:
|
||||
|
||||
1. They're trying to solve their own problem
|
||||
2. They believe that their solution will help others
|
||||
|
||||
Therefore our responsibility is to:
|
||||
|
||||
1. Determine if there's any reason the PR **shouldn't** go through
|
||||
* (i.e. it's out of line with the project goals)
|
||||
2. Help the contributor get 80% of the way there
|
||||
* Determine criteria that need to be met to move forward
|
||||
3. Make the last few changes, if needed (rename a thing, fix a space)
|
||||
|
||||
|
||||
|
||||
|
||||
## 2. Opinions are the Burden of the Bearer
|
||||
|
||||
## 3. Remain Politically Neutral
|
||||
|
||||
First off, we recognize that everyone has moral (and amoral) values that they feel strongly about and want to enforce among other people. This is a natural human tendency. That doesn't make mud-slinging okay, but it does make it **forgivable**.
|
||||
|
||||
|
||||
#### Erring on the side of Hypocrisy
|
||||
|
||||
To both hold a moral view, which morality dictates _should_ be enforced, _and_ to accept others who oppose that view is paroxical - therefore we err on the side of **hypocrisy**:
|
||||
|
||||
It is better to **accept** than to oppose.
|
||||
|
||||
Being **obnoxious** makes it difficult for others to accept you.
|
||||
|
||||
Just as arsenic and selenium (which are deadly in high doses) are
|
||||
[nutrients](https://www.ncbi.nlm.nih.gov/pmc/articles/PMC2246629/) in microdoses,
|
||||
we view accepting, correcting, and forgiving a little toxicity to be more in line
|
||||
with our values of tolerance than blocking.
|
||||
|
||||
#### Safe to be Wrong
|
||||
|
||||
We value **individuality** (it comes with the name).
|
||||
|
||||
This is a **safe** space to have an opinion and to express it, even if it possibly offends someone,
|
||||
as long as you're also open to feedback and you **don't become obnoxious**.
|
||||
|
||||
That said, please don't preach political views or mud-sling. Incidental conversation is fine. We'll block obnoxious people.
|
||||
|
||||
#### Our Politics
|
||||
|
||||
We are Root. All on it's own that expression, "We are Root", carries a lot of political bias - we care about individual ownership, control, identity, minimalism, etc. Creating a community that has those interests and values in common has a polarizing affect. Therefore we can't be perfectly nuetral. In the absence of perfect nuetrality, we strive for an "it is what it is" mentality.
|
Loading…
Reference in New Issue