make lint fails with pep8 1.6.0 (2015-02-06)
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Charm Helpers |
Fix Released
|
High
|
Billy Olsen |
Bug Description
A new version of pep8 (1.6.0) was recently released which introduces some new checks. There are various violations of the new rule checks for E402 and E731.
Some examples of this are as follows:
wolsen@
Checking for Python syntax...
charmhelpers/
charmhelpers/
charmhelpers/
charmhelpers/
charmhelpers/
charmhelpers/
charmhelpers/
...
These need to be cleaned up.
Related branches
- Jorge Niedbalski (community): Approve
-
Diff: 712 lines (+101/-110)24 files modifiedcharmhelpers/contrib/ansible/__init__.py (+4/-1)
charmhelpers/contrib/charmhelpers/__init__.py (+18/-15)
charmhelpers/contrib/network/ufw.py (+2/-3)
charmhelpers/contrib/python/debug.py (+2/-2)
charmhelpers/contrib/python/packages.py (+2/-2)
charmhelpers/contrib/python/rpdb.py (+3/-3)
charmhelpers/contrib/python/version.py (+2/-2)
charmhelpers/core/fstab.py (+2/-2)
charmhelpers/core/sysctl.py (+2/-2)
charmhelpers/core/unitdata.py (+1/-1)
charmhelpers/fetch/archiveurl.py (+10/-10)
charmhelpers/fetch/giturl.py (+1/-1)
tests/contrib/charmhelpers/test_charmhelpers.py (+21/-37)
tests/contrib/database/test_mysql.py (+1/-1)
tests/contrib/network/test_ufw.py (+2/-1)
tests/contrib/openstack/test_os_templating.py (+6/-6)
tests/contrib/python/test_debug.py (+2/-2)
tests/contrib/python/test_packages.py (+2/-2)
tests/contrib/python/test_version.py (+2/-2)
tests/core/test_fstab.py (+2/-2)
tests/core/test_hookenv.py (+2/-1)
tests/core/test_sysctl.py (+2/-2)
tests/fetch/test_archiveurl.py (+8/-8)
tests/tools/test_charm_helper_sync.py (+2/-2)
Changed in charm-helpers: | |
assignee: | nobody → Billy Olsen (billy-olsen) |
status: | New → In Progress |
Changed in charm-helpers: | |
importance: | Undecided → High |
status: | In Progress → Fix Released |