From 121dbe69252009c2d37a3b9d39c19823c4f20c6f Mon Sep 17 00:00:00 2001
From: Rafael Guterres Jeffman <rjeffman@redhat.com>
Date: Thu, 29 Apr 2021 19:07:16 -0300
Subject: [PATCH] Fix pylint warning `consider-merging-isinstance`.
---
plugins/modules/ipahost.py | 2 +-
plugins/modules/ipauser.py | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/plugins/modules/ipahost.py b/plugins/modules/ipahost.py
index 5864e180..0bd5b1b4 100644
--- a/plugins/modules/ipahost.py
+++ b/plugins/modules/ipahost.py
@@ -862,7 +862,7 @@ def main():
ok_to_auth_as_delegate, force, reverse, ip_address,
update_dns, update_password)
- elif isinstance(host, str) or isinstance(host, unicode):
+ elif isinstance(host, (str, unicode)):
name = host
else:
ansible_module.fail_json(msg="Host '%s' is not valid" %
diff --git a/plugins/modules/ipauser.py b/plugins/modules/ipauser.py
index 23b9710e..0a104cb9 100644
--- a/plugins/modules/ipauser.py
+++ b/plugins/modules/ipauser.py
@@ -1029,7 +1029,7 @@ def main():
email = extend_emails(email, default_email_domain)
- elif isinstance(user, str) or isinstance(user, unicode):
+ elif isinstance(user, (str, unicode)):
name = user
else:
ansible_module.fail_json(msg="User '%s' is not valid" %
--
GitLab