openstack
/
tempest
Code Issues Proposed changes

Add deprecated code review guideline 

This commit adds a deprecated code review guideline. We have some
deprecated code (e.g. stress test) now. And we sometimes wonder whether
we should review a patch for deprecated code or not. So this patch tries
to clear up the situation.

Change-Id: I188cd9fccae9396b143fa85071387e8a5bb5f3d1
This commit is contained in:
Masayuki Igawa 2016-09-02 16:38:56 +09:00
parent 29741bc7c2
commit 4661706f68
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
REVIEWING.rst
View File

@ -86,6 +86,12 @@ something extra.
.. _reno: http://docs.openstack.org/developer/reno/
Deprecated Code
---------------
Sometimes we have some bugs in deprecated code. Basically, we leave it. Because
we don't need to maintain it. However, if the bug is critical, we might need to
fix it. When it will happen, we will deal with it on a case-by-case basis.
When to approve
---------------
* Every patch needs two +2s before being approved.