review.fuel-infra Code Review - openstack-build/cinder-build.git/commit

]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit

Code Review / openstack-build / cinder-build.git / commit

author	Christian Berendt <berendt@b1-systems.de>
	Fri, 1 Aug 2014 11:10:20 +0000 (13:10 +0200)
committer	Christian Berendt <berendt@b1-systems.de>
	Fri, 1 Aug 2014 12:34:47 +0000 (14:34 +0200)
commit	1be1da1020281a83d6285173a6d6078ffe979194
tree	8e6e3642265d5503012f1e68ecd182ef1c665947	tree \| snapshot
parent	3a04abfc5c9cd06ee229afd8bd4da4d932dd5244	commit \| diff

Enable checks for E711, E712 and E713

* E711 comparison to None should be 'if cond is None:'
* E712 comparison to True should be 'if cond is True:' or 'if cond:'
* E713 test for membership should be 'not in'

Change-Id: I1481209a85fa6db0343d0111e2b19d9fab4538a7

12 files changed:

RSS Atom

bin/cinder-rtstool		diff \| blob \| history
cinder/db/sqlalchemy/api.py		diff \| blob \| history
cinder/tests/api/fakes.py		diff \| blob \| history
cinder/tests/api/v2/test_volumes.py		diff \| blob \| history
cinder/tests/keymgr/test_key.py		diff \| blob \| history
cinder/tests/test_hds_iscsi.py		diff \| blob \| history
cinder/utils.py		diff \| blob \| history
cinder/volume/api.py		diff \| blob \| history
cinder/volume/drivers/emc/emc_smis_common.py		diff \| blob \| history
cinder/volume/drivers/eqlx.py		diff \| blob \| history
cinder/volume/drivers/netapp/iscsi.py		diff \| blob \| history
tox.ini		diff \| blob \| history