[telemetry] Procedure for merging patches
Trinh Nguyen
dangtrinhnt at gmail.com
Thu Apr 11 07:35:53 UTC 2019
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>*
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-discuss/attachments/20190411/4a53e405/attachment.html>
More information about the openstack-discuss
mailing list