a-c-m
/
community.crypto
mirror of https://github.com/ansible-collections/community.crypto.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix some new linting errors (#499) 

* Fix some new linting errors.

* More linting errors.

ci_complete
pull/500/head
Felix Fontein 2022-08-12 08:34:51 +02:00 committed by GitHub
parent c0bab015a4
commit fd1263c9aa
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
plugins/module_utils/acme/acme.py
View File

@ -48,6 +48,7 @@ except ImportError:
IPADDRESS_IMPORT_ERROR = traceback.format_exc() IPADDRESS_IMPORT_ERROR = traceback.format_exc()
else: else:
HAS_IPADDRESS = True HAS_IPADDRESS = True
IPADDRESS_IMPORT_ERROR = None
def _assert_fetch_url_success(module, response, info, allow_redirect=False, allow_client_error=True, allow_server_error=True): def _assert_fetch_url_success(module, response, info, allow_redirect=False, allow_client_error=True, allow_server_error=True):

6
plugins/plugin_utils/action_module.py
View File

@ -217,7 +217,7 @@ class AnsibleActionModule(object):
# this uses exceptions as it happens before we can safely call fail_json # this uses exceptions as it happens before we can safely call fail_json
alias_warnings = [] alias_warnings = []
alias_results, self._legal_inputs = handle_aliases(spec, param, alias_warnings=alias_warnings) alias_results, self._legal_inputs = handle_aliases(spec, param, alias_warnings=alias_warnings) # pylint: disable=used-before-assignment
for option, alias in alias_warnings: for option, alias in alias_warnings:
self.warn('Both option %s and its alias %s are set.' % (option_prefix + option, option_prefix + alias)) self.warn('Both option %s and its alias %s are set.' % (option_prefix + option, option_prefix + alias))
@ -241,12 +241,12 @@ class AnsibleActionModule(object):
param = self.params param = self.params
try: try:
self.no_log_values.update(list_no_log_values(spec, param)) self.no_log_values.update(list_no_log_values(spec, param)) # pylint: disable=used-before-assignment
except TypeError as te: except TypeError as te:
self.fail_json(msg="Failure when processing no_log parameters. Module invocation will be hidden. " self.fail_json(msg="Failure when processing no_log parameters. Module invocation will be hidden. "
"%s" % to_native(te), invocation={'module_args': 'HIDDEN DUE TO FAILURE'}) "%s" % to_native(te), invocation={'module_args': 'HIDDEN DUE TO FAILURE'})
for message in list_deprecations(spec, param): for message in list_deprecations(spec, param): # pylint: disable=used-before-assignment
self.deprecate(message['msg'], version=message.get('version'), date=message.get('date'), self.deprecate(message['msg'], version=message.get('version'), date=message.get('date'),
collection_name=message.get('collection_name')) collection_name=message.get('collection_name'))

1
tests/ee/roles/smoke/library/smoke_ipaddress.py
View File

@ -31,6 +31,7 @@ from ansible.module_utils.basic import AnsibleModule, missing_required_lib
try: try:
import ipaddress import ipaddress
HAS_IPADDRESS = True HAS_IPADDRESS = True
IPADDRESS_IMP_ERR = None
except ImportError as exc: except ImportError as exc:
IPADDRESS_IMP_ERR = traceback.format_exc() IPADDRESS_IMP_ERR = traceback.format_exc()
HAS_IPADDRESS = False HAS_IPADDRESS = False

1
tests/ee/roles/smoke/library/smoke_pyyaml.py
View File

@ -31,6 +31,7 @@ from ansible.module_utils.basic import AnsibleModule, missing_required_lib
try: try:
import yaml import yaml
HAS_PYYAML = True HAS_PYYAML = True
PYYAML_IMP_ERR = None
except ImportError as exc: except ImportError as exc:
PYYAML_IMP_ERR = traceback.format_exc() PYYAML_IMP_ERR = traceback.format_exc()
HAS_PYYAML = False HAS_PYYAML = False