openstack/ansible-collections-openstack
Code Issues Proposed changes

Fix F401 and remove exclusion

Browse Source 
Change-Id: Id2dbff06925ed8326e4a214aabc19d4ada5db69c
This commit is contained in:
Monty Taylor 2019-12-09 13:23:15 -05:00
parent 92d7d7caeb
commit 86670bcec9
7 changed files with 15 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
plugins/modules/os_networks_info.py
View File

@ -125,9 +125,10 @@ openstack_networks:
'''
from ansible.module_utils.basic import AnsibleModule
from ansible_collections.openstack.cloud.plugins.module_utils.openstack import (openstack_full_argument_spec,
openstack_module_kwargs,
openstack_cloud_from_module)
from ansible_collections.openstack.cloud.plugins.module_utils.openstack import (
openstack_full_argument_spec,
openstack_cloud_from_module,
)
def main():

7
plugins/modules/os_quota.py
View File

@ -236,9 +236,10 @@ except ImportError:
HAS_KEYSTONEAUTH1 = False
from ansible.module_utils.basic import AnsibleModule, missing_required_lib
from ansible_collections.openstack.cloud.plugins.module_utils.openstack import (openstack_full_argument_spec,
openstack_module_kwargs,
openstack_cloud_from_module)
from ansible_collections.openstack.cloud.plugins.module_utils.openstack import (
openstack_full_argument_spec,
openstack_cloud_from_module,
)
def _get_volume_quotas(cloud, project):

2
plugins/modules/os_user.py
View File

@ -135,8 +135,6 @@ user:
type: str
sample: "demouser"
'''
from distutils.version import StrictVersion
from ansible.module_utils.basic import AnsibleModule
from ansible_collections.openstack.cloud.plugins.module_utils.openstack import (openstack_full_argument_spec,
openstack_module_kwargs,

7
plugins/modules/os_user_info.py
View File

@ -118,9 +118,10 @@ openstack_users:
'''
from ansible.module_utils.basic import AnsibleModule
from ansible_collections.openstack.cloud.plugins.module_utils.openstack import (openstack_full_argument_spec,
openstack_module_kwargs,
openstack_cloud_from_module)
from ansible_collections.openstack.cloud.plugins.module_utils.openstack import (
openstack_full_argument_spec,
openstack_cloud_from_module,
)
def main():

3
plugins/modules/os_volume.py
View File

@ -99,9 +99,6 @@ volume:
type: dict
sample: {'...'}
'''
from distutils.version import StrictVersion
from ansible.module_utils.basic import AnsibleModule
from ansible_collections.openstack.cloud.plugins.module_utils.openstack import (openstack_full_argument_spec,
openstack_module_kwargs,

2
tests/unit/compat/builtins.py
View File

@ -26,7 +26,7 @@ __metaclass__ = type
# One unittest needs to import builtins via __import__() so we need to have
# the string that represents it
try:
import __builtin__
import __builtin__ # noqa
except ImportError:
BUILTINS = 'builtins'
else:

4
tox.ini
View File

@ -49,9 +49,9 @@ commands = {posargs}
# Also, both openstacksdk and Donald Knuth disagree with the rule. Line
# breaks should occur before the binary operator for readability.
# H4 are rules for docstrings. Maybe we should clean them?
# E501,E402,H301,H236,F401 are ignored so we can import the existing
# E501,E402,H301,H236 are ignored so we can import the existing
# modules unchanged and then clean them in subsequent patches.
ignore = W503,H4,E501,E402,H301,H236,F401
ignore = W503,H4,E501,E402,H301,H236
show-source = True
exclude=.venv,.git,.tox,dist,doc,*lib/python*,*egg,build,ansible_collections