Drop max-complexity to 47

Code Issues Proposed changes

Now that we fixed the two worst offenders, drop the mccabe
max-complexity setting in flake8 to 47.
Add note explaining how to generate a list of functions that are too
'complex'.
Change-Id: I44000e9e95c457ae9000c428460b5825d8ad3f92

This commit is contained in:

Joe Gordon

2014年10月29日 12:22:29 -07:00

committed by Matt Riedemann

parent 3adb279913

commit f455cbcbcb

1 changed files with 4 additions and 2 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download Patch File Download Diff File Expand all files Collapse all files

6
tox.ini

View File

@@ -60,9 +60,11 @@ commands = python setup.py build_sphinx

ignore = E121,E122,E123,E124,E125,E126,E127,E128,E129,E131,E251,H405,H803,H904

exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,tools

# 67 is currently the most complex thing we have

# To get a list of functions that are more complex than 25, set max-complexity

# to 25 and run 'tox -epep8'.

# 46 is currently the most complex thing we have

# TODO(jogo): get this number down to 25 or so

max-complexity=68

max-complexity=47

[hacking]

local-check-factory = nova.hacking.checks.factory

Drop max-complexity to 47

6 tox.ini Unescape Escape View File

6
tox.ini

View File