diff options
author | Amrith Kumar <amrith@tesora.com> | 2016-07-31 07:32:28 -0400 |
---|---|---|
committer | Amrith Kumar <amrith@tesora.com> | 2016-07-31 07:33:27 -0400 |
commit | ddac516c85e18077fe3a6184d2505c508219d4f9 (patch) | |
tree | 9952937106999e03c3b4c79b27ad795ac206ee4f /CONTRIBUTING.rst | |
parent | 67940e90a68250f712e2dc753414b5ed2c7bfd9a (diff) | |
download | trove-ddac516c85e18077fe3a6184d2505c508219d4f9.tar.gz |
Update CONTRIBUTING.rst
Updated CONTRIBUTING.rst to reflect discussions at mid-cycle regarding
abandonment of changes that are languishing in the gerrit review
system.
Change-Id: Ie9d22b01d09c2cdb250abd0596c0216ec457df4d
Diffstat (limited to 'CONTRIBUTING.rst')
-rw-r--r-- | CONTRIBUTING.rst | 31 |
1 files changed, 29 insertions, 2 deletions
diff --git a/CONTRIBUTING.rst b/CONTRIBUTING.rst index 14b7b7bd..f442a8ce 100644 --- a/CONTRIBUTING.rst +++ b/CONTRIBUTING.rst @@ -31,8 +31,11 @@ wiki <http://wiki.openstack.org>`_, blogs, or on IRC at ``#openstack-trove`` on ``irc.freenode.net``. +House Rules +=========== + Code Reviews -============ +------------ We value your contribution in reviewing code changes submitted by others, as this helps increase the quality of the product as well. @@ -88,7 +91,7 @@ Other references: - https://review.openstack.org/#/c/116176/ Approving changes -================= +----------------- The Trove project follows the conventions below in approving changes. @@ -116,6 +119,27 @@ additional negative comments by a certain time definite. We will however still require that at least one other person review (and +1 or +2) the change before it can be +A'ed. +Abandoning changes +------------------ + +At the Trove mid-cycle held in July 2016 we discussed our process for +abandoning changes and concluded that we would adopt the following +process. + +1. We will take a more proactive policy towards abandoning changes + that have not been merged for a long time. + +2. A list of changes proposed for abandonment will be presented at a + weekly meeting and if there is no objection, those changes will be + abandoned. If the patch sets are associated with bugs, the bugs + will be unassigned. + +3. In general, changes will be proposed for abandonment if the change + being proposed has either been addressed in some other patch set, + or if the patch is not being actively maintained by the author and + there is no available volunteer who will step up to take over the + patch set. + Trove Documentation =================== @@ -173,6 +197,9 @@ tox will use the `openstack-doc-tools package these tests. openstack-doc-tools has a requirement on maven for the build check. +Testing +======= + Usage for integration testing ----------------------------- |