[airflow] branch master updated: Moves Commiter's guide to CONTRIBUTING.rst (#11314)

[po...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/master by this push:
     new d404cb0  Moves Commiter's guide to CONTRIBUTING.rst (#11314)
d404cb0 is described below

commit d404cb06dd52d391a9ef26ef700b61f04a771cd5
Author: Jarek Potiuk <ja...@polidea.com>
AuthorDate: Wed Oct 7 21:14:55 2020 +0200

    Moves Commiter's guide to CONTRIBUTING.rst (#11314)
    
    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
---
 CONTRIBUTING.rst | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/CONTRIBUTING.rst b/CONTRIBUTING.rst
index 81e07fc..1976a82 100644
--- a/CONTRIBUTING.rst
+++ b/CONTRIBUTING.rst
@@ -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/>`__