|
| 1 | +Symfony Core Team |
| 2 | +================= |
| 3 | + |
| 4 | +This document states the rules that govern the Symfony Core group. These rules |
| 5 | +are effective upon publication of this document and all Symfony Core members |
| 6 | +must adhere to said rules and protocol. |
| 7 | + |
| 8 | +Core Organization |
| 9 | +----------------- |
| 10 | + |
| 11 | +Symfony Core members are divided into three groups. Each member can only belong |
| 12 | +to one group at a time. The privileges granted to a group are automatically |
| 13 | +granted to all higher priority groups. |
| 14 | + |
| 15 | +The Symfony Core groups, in descending order of priority, are as follows: |
| 16 | + |
| 17 | +1. **Project Leader** |
| 18 | + |
| 19 | +* Elects members in any other group; |
| 20 | +* Merges pull requests in all Symfony repositories. |
| 21 | + |
| 22 | +2. **Mergers** |
| 23 | + |
| 24 | +* Merge pull requests for the component or components on which they have been |
| 25 | + granted privileges. |
| 26 | + |
| 27 | +3. **Deciders** |
| 28 | + |
| 29 | +* Decide to merge or reject a pull request. |
| 30 | + |
| 31 | +Active Core Members |
| 32 | +~~~~~~~~~~~~~~~~~~~ |
| 33 | + |
| 34 | +..role::leader |
| 35 | +..role::merger |
| 36 | +..role::decider |
| 37 | + |
| 38 | +* **Project Leader**: |
| 39 | + |
| 40 | + * **Fabien Potencier** (:leader:`fabpot`). |
| 41 | + |
| 42 | +* **Mergers**: |
| 43 | + |
| 44 | + * **Bernhard Schussek** (:merger:`webmozart`) can merge into theForm_, |
| 45 | +Validator_,Icu_,Intl_,Locale_,OptionsResolver_ andPropertyAccess_ |
| 46 | + components; |
| 47 | + |
| 48 | + * **Tobias Schultze** (:merger:`Tobion`) can merge into theRouting_ |
| 49 | + component; |
| 50 | + |
| 51 | + * **Romain Neutron** (:merger:`romainneutron`) can merge into the |
| 52 | +Process_ component; |
| 53 | + |
| 54 | + * **Nicolas Grekas** (:merger:`nicolas-grekas`) can merge into theDebug_ |
| 55 | + component. |
| 56 | + |
| 57 | +* **Deciders**: |
| 58 | + |
| 59 | + * **Christophe Coevoet** (:decider:`stof`); |
| 60 | + * **Jakub Zalas** (:decider:`jakzal`); |
| 61 | + * **Jordi Boggiano** (:decider:`seldaek`); |
| 62 | + * **Lukas Kahwe Smith** (:decider:`lsmith77`). |
| 63 | + |
| 64 | +Core Membership Application |
| 65 | +~~~~~~~~~~~~~~~~~~~~~~~~~~~ |
| 66 | + |
| 67 | +At present, new Symfony Core membership applications are not accepted. |
| 68 | + |
| 69 | +Core Membership Revocation |
| 70 | +~~~~~~~~~~~~~~~~~~~~~~~~~~ |
| 71 | + |
| 72 | +A Symfony Core membership can be revoked for any of the following reasons: |
| 73 | + |
| 74 | +* Refusal to follow the rules and policies stated in this document; |
| 75 | +* Lack of activity for the past six months; |
| 76 | +* Willful negligence or intent to harm the Symfony project; |
| 77 | +* Upon decision of the **Project Leader**. |
| 78 | + |
| 79 | +Should new Symfony Core memberships be accepted in the future, revoked |
| 80 | +members must wait at least 12 months before re-applying. |
| 81 | + |
| 82 | +Code Development Rules |
| 83 | +---------------------- |
| 84 | + |
| 85 | +Symfony project development is based on pull requests proposed by any member |
| 86 | +of the Symfony community. Pull request acceptance or rejection is decided based |
| 87 | +on the votes cast by the Symfony Core members. |
| 88 | + |
| 89 | +Pull Request Voting Policy |
| 90 | +~~~~~~~~~~~~~~~~~~~~~~~~~~ |
| 91 | + |
| 92 | +* ``-1`` votes must always be justified by technical and objective reasons; |
| 93 | + |
| 94 | +* ``+1`` votes do not require justification, unless there is at least one |
| 95 | + ``-1`` vote; |
| 96 | + |
| 97 | +* Core members can change their votes as many times as they desire |
| 98 | + during the course of a pull request discussion; |
| 99 | + |
| 100 | +* Core members are not allowed to vote on their own pull requests. |
| 101 | + |
| 102 | +Pull Request Merging Policy |
| 103 | +~~~~~~~~~~~~~~~~~~~~~~~~~~~ |
| 104 | + |
| 105 | +A pull request **can be merged** if: |
| 106 | + |
| 107 | +* Enough time was given for peer reviews (a few minutes for typos or minor |
| 108 | + changes, at least 2 days for "regular" pull requests, and 4 days for pull |
| 109 | + requests with "a significant impact"); |
| 110 | + |
| 111 | +* It is a minor change [1]_, regardless of the number of votes; |
| 112 | + |
| 113 | +* At least the component's **Merger** or two other Core members voted ``+1`` |
| 114 | + and no Core member voted ``-1``. |
| 115 | + |
| 116 | +Pull Request Merging Process |
| 117 | +~~~~~~~~~~~~~~~~~~~~~~~~~~~~ |
| 118 | + |
| 119 | +All code must be committed to the repository through pull requests, except for |
| 120 | +minor changes [1]_ which can be committed directly to the repository. |
| 121 | + |
| 122 | +**Mergers** must always use the command-line ``gh`` tool provided by the |
| 123 | +**Project Leader** to merge the pull requests. |
| 124 | + |
| 125 | +Release Policy |
| 126 | +~~~~~~~~~~~~~~ |
| 127 | + |
| 128 | +The **Project Leader** is also the release manager for every Symfony version. |
| 129 | + |
| 130 | +Symfony Core Rules and Protocol Amendments |
| 131 | +------------------------------------------ |
| 132 | + |
| 133 | +The rules described in this document may be amended at anytime at the |
| 134 | +discretion of the **Project Leader**. |
| 135 | + |
| 136 | + |
| 137 | +.. [1]Minor changes comprise typos, DocBlock fixes, code standards |
| 138 | + violations, and minor CSS, JavaScript and HTML modifications. |
| 139 | +
|
| 140 | +.. _Form:https://github.com/symfony/Form |
| 141 | +.. _Validator:https://github.com/symfony/Validator |
| 142 | +.. _Icu:https://github.com/symfony/Icu |
| 143 | +.. _Intl:https://github.com/symfony/Intl |
| 144 | +.. _Locale:https://github.com/symfony/Locale |
| 145 | +.. _OptionsResolver:https://github.com/symfony/OptionsResolver |
| 146 | +.. _PropertyAccess:https://github.com/symfony/PropertyAccess |
| 147 | +.. _Routing:https://github.com/symfony/Routing |
| 148 | +.. _Process:https://github.com/symfony/Process |
| 149 | +.. _Debug:https://github.com/symfony/Debug |