Document the blacklist intent for reviewers.

Change-Id: If99e6997b11e50758a9041cacbc4f4c7938cb11f
This commit is contained in:
Robert Collins 2015-10-20 10:52:27 +13:00
parent 5d15dfa692
commit 7eab32a33b
2 changed files with 14 additions and 0 deletions

View File

@ -236,6 +236,17 @@ General Review Criteria
blueprint requires the new specification. Ideally, changes should blueprint requires the new specification. Ideally, changes should
already be proposed, so that its use can be seen. already be proposed, so that its use can be seen.
- The blacklist is for handling dependencies that cannot be constrained.
For instance, linters which each project has at a different release level,
and which make projects fail on every release (because they add rules) -
those cannot be globally constrained unless we coordinate updating all of
OpenStack to the new release at the same time - but given the volunteer
and loosely coupled nature of the big tent that is infeasible. Dependencies
that are only used in unconstrained places should not be blacklisted - they
may be constrained in future, and there's no harm caused by constraining
them today. Entries in the blacklist should have a comment explaining the
reason for blacklisting.
For new Requirements For new Requirements
-------------------- --------------------

View File

@ -1,7 +1,10 @@
# linters - each project may hve a different version with loose convergence
# over time.
flake8 flake8
hacking hacking
mccabe mccabe
pep8 pep8
pyflakes pyflakes
pylint pylint
# release changelog generator - xxx - under discussion
reno reno