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
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
--------------------

View File

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