[telemetry] Procedure for merging patches
Rong Zhu
aaronzhu1121 at gmail.com
Thu Apr 11 07:46:53 UTC 2019
Hi,
As usual, There are have +2s, then the next core can +2/A, why we need PTL
final +A?
Thanks,
Rong Zhu
Trinh Nguyen <dangtrinhnt at gmail.com>于2019年4月11日 周四15:39写道:
> Hi team,
>
> One of the thing that has been brought up in the last meeting is what is
> the process of merging a patch. So, let's try this first:
>
> - For small patches (e.g., change strings, fix typos, update docs):
> any core can approve for merging.
> - For a patch that implements a new feature, removes a feature, patch
> that has huge changes (in the number of changed files and loc, etc.) or any
> patch that is in doubt: we need at least 2 cores to give a +2 and then the
> PTL will merge it.
>
> Of course, we can change the procedure when we see issues. For now, this
> should be a guideline for the team.
>
> Please feel free to comment.
>
> Yours,
>
> --
> *Trinh Nguyen*
> *www.edlab.xyz <https://www.edlab.xyz>*
>
> --
Thanks,
Rong Zhu
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-discuss/attachments/20190411/86e2aefe/attachment-0001.html>
More information about the openstack-discuss
mailing list