diff --git a/HACKING.rst b/HACKING.rst index 8744a93ac4..27c5e4851b 100644 --- a/HACKING.rst +++ b/HACKING.rst @@ -5,6 +5,17 @@ OpenStack Style Commandments http://docs.openstack.org/developer/hacking/ - Step 2: Read on +Deviations from Global Hacking +------------------------------ + +The following checks are specifically skipped in OpenStackClient:: + +* H305 - py2/py3 compatibility problems +* H307 - py2/py3 compatibility problems +* H402 - one line docstring ends in period +* H904 - backslash continuation line + + General ------- - thou shalt not violate causality in our time cone, or else diff --git a/openstackclient/common/restapi.py b/openstackclient/common/restapi.py index f20ad23dd2..a4822a1028 100644 --- a/openstackclient/common/restapi.py +++ b/openstackclient/common/restapi.py @@ -20,9 +20,9 @@ import logging import requests try: - from urllib.parse import urlencode + from urllib.parse import urlencode # noqa except ImportError: - from urllib import urlencode + from urllib import urlencode # noqa USER_AGENT = 'RAPI' diff --git a/openstackclient/object/v1/lib/container.py b/openstackclient/object/v1/lib/container.py index 0bae23493f..72e97d4e1d 100644 --- a/openstackclient/object/v1/lib/container.py +++ b/openstackclient/object/v1/lib/container.py @@ -17,9 +17,9 @@ """Object v1 API library""" try: - from urllib.parse import urlparse + from urllib.parse import urlparse # noqa except ImportError: - from urlparse import urlparse + from urlparse import urlparse # noqa def list_containers( diff --git a/openstackclient/object/v1/lib/object.py b/openstackclient/object/v1/lib/object.py index 646737bdbd..ffc7e9b8ed 100644 --- a/openstackclient/object/v1/lib/object.py +++ b/openstackclient/object/v1/lib/object.py @@ -19,9 +19,9 @@ import six try: - from urllib.parse import urlparse + from urllib.parse import urlparse # noqa except ImportError: - from urlparse import urlparse + from urlparse import urlparse # noqa def list_objects( diff --git a/tox.ini b/tox.ini index bdb98bec69..b6935c2fb4 100644 --- a/tox.ini +++ b/tox.ini @@ -29,6 +29,6 @@ commands= python setup.py build_sphinx [flake8] -ignore = H302,H305,H307,H402,H904 +ignore = H305,H307,H402,H904 show-source = True exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,tools