diff options
author | Martin Basti <mbasti@redhat.com> | 2016-09-26 18:24:39 +0200 |
---|---|---|
committer | Martin Basti <mbasti@redhat.com> | 2016-09-27 13:35:58 +0200 |
commit | 45e3aee35219c89c07d590003a334f8db658a3b2 (patch) | |
tree | cb535e7f725f31d0bc6b5faf405b08fac05a3e96 /client | |
parent | 9d83be3647547cfca4e129cfeb63771213232cf7 (diff) | |
download | freeipa-45e3aee35219c89c07d590003a334f8db658a3b2.tar.gz freeipa-45e3aee35219c89c07d590003a334f8db658a3b2.tar.xz freeipa-45e3aee35219c89c07d590003a334f8db658a3b2.zip |
Pylint: enable check for unused-variables
Unused variables may:
* make code less readable
* create dead code
* potentialy hide issues/errors
Enabled check should prevent to leave unused variable in code
Check is locally disabled for modules that fix is not clear or easy or have too many occurences of
unused variables
Reviewed-By: Florence Blanc-Renaud <frenaud@redhat.com>
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
Diffstat (limited to 'client')
-rwxr-xr-x | client/ipa-client-automount | 1 | ||||
-rwxr-xr-x | client/ipa-client-install | 2 |
2 files changed, 3 insertions, 0 deletions
diff --git a/client/ipa-client-automount b/client/ipa-client-automount index b444d3ea5..91bdc88f4 100755 --- a/client/ipa-client-automount +++ b/client/ipa-client-automount @@ -45,6 +45,7 @@ from ipaplatform.tasks import tasks from ipaplatform import services from ipaplatform.paths import paths +# pylint: disable=unused-variable def parse_options(): usage = "%prog [options]\n" diff --git a/client/ipa-client-install b/client/ipa-client-install index 3614af871..d38eaf951 100755 --- a/client/ipa-client-install +++ b/client/ipa-client-install @@ -72,6 +72,8 @@ error was: """ % e, file=sys.stderr) sys.exit(1) +# pylint: disable=unused-variable + SUCCESS = 0 CLIENT_INSTALL_ERROR = 1 CLIENT_NOT_CONFIGURED = 2 |