From 686f876b026d3b7752b14469b1ab6dc328fc1a06 Mon Sep 17 00:00:00 2001 From: Tony Breeds Date: Wed, 31 Oct 2018 08:54:30 +1100 Subject: [PATCH] Removed unused err variable Frankly I don't know why this is now a problem but pep8 is failing[1] with: cliff/app.py:403:9: F841 local variable 'err' is assigned to but never used It looks to me like the 'err' in the except path is causing this to be shadowed and unused? [1] http://logs.openstack.org/18/614218/2/check/openstack-tox-pep8/9930eb3/job-output.txt.gz#_2018-10-30_16_34_38_436438 Change-Id: I4760942aef3e41fbaeabb8e88ac198fff3dfb709 --- cliff/app.py | 1 - 1 file changed, 1 deletion(-) diff --git a/cliff/app.py b/cliff/app.py index f490a5ca..94b3d76a 100644 --- a/cliff/app.py +++ b/cliff/app.py @@ -389,7 +389,6 @@ class App(object): if 'cmd_name' in utils.getargspec(cmd_factory.__init__).args: kwargs['cmd_name'] = cmd_name cmd = cmd_factory(self, self.options, **kwargs) - err = None result = 1 try: self.prepare_to_run_command(cmd)