Skip to content

Commit

Permalink
Moves Commiter's guide to CONTRIBUTING.rst (#11314)
Browse files Browse the repository at this point in the history
I decided to move it to CONTRIBUTING.rst as is it is an important
documentation on what policies we have agreed to as community and
also it is a great resource for the contributor to learn what are
the committer's responsibilities.

Fixes: #10179
  • Loading branch information
potiuk committed Oct 7, 2020
1 parent fe59f26 commit d404cb0
Showing 1 changed file with 28 additions and 0 deletions.
28 changes: 28 additions & 0 deletions CONTRIBUTING.rst
Original file line number Diff line number Diff line change
Expand Up @@ -1158,6 +1158,34 @@ Here are a few rules that are important to keep in mind when you enter our commu
* It’s OK to express your own emotions while communicating - it helps other people to understand you
* Be considerate for feelings of others. Tell about how you feel not what you think of others

Committer Responsibilities
==========================

Committers are more than contributors. While it's important for committers to maintain standing by
committing code, their key role is to build and foster a healthy and active community.
This means that committers should:

* Review PRs in a timely and reliable fashion
* They should also help to actively whittle down the PR backlog
* Answer questions (i.e. on the dev list, in PRs, in Github Issues, slack, etc...)
* Take on core changes/bugs/feature requests
* Some changes are important enough that a committer needs to ensure it gets done. This is especially
the case if no one from the community is taking it on.
* Improve processes and tooling
* Refactoring code

Commit Policy
=============

The following commit policy passed by a vote 8(binding FOR) to 0 against on May 27, 2016 on the dev list
and slightly modified and consensus reached in October 2020:

* Commits need a +1 vote from a committer who is not the author
* Furthermore, do not merge a PR that regresses linting or does not pass CI tests (unless we have
justification such as clearly transient error).
* When we do AIP voting, both PMC and committer +1s are considered as binding vote based on internal
PMC discussion.

Resources & Links
=================
- `Airflow’s official documentation <http://airflow.apache.org/>`__
Expand Down

0 comments on commit d404cb0

Please sign in to comment.