openstack/python-magnumclient
Code Issues Proposed changes

Rename variables to address pep8 error

Browse Source 
Without rename, we see this:
    E741 ambiguous variable name 'l'

Change-Id: Ic1d357db594dd6f8f8db162033b6cf408e84453c
Story: 2007515
Task: 39775
This commit is contained in:
Bharat Kunwar 2020-05-14 11:33:50 +00:00
parent 26c08b9317
commit 1c1cfd7109
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
magnumclient/common/cliutils.py
View File

@ -404,8 +404,8 @@ def get_service_type(f):
return getattr(f, 'service_type', None)
def pretty_choice_list(l):
return ', '.join("'%s'" % i for i in l)
def pretty_choice_list(lst):
return ', '.join("'%s'" % i for i in lst)
def exit(msg=''):

6
magnumclient/common/utils.py
View File

@ -118,12 +118,12 @@ def format_labels(lbls, parse_comma=True):
lbls = lbls[0].replace(';', ',').split(',')
labels = {}
for l in lbls:
for lbl in lbls:
try:
(k, v) = l.split(('='), 1)
(k, v) = lbl.split(('='), 1)
except ValueError:
raise exc.CommandError(_('labels must be a list of KEY=VALUE '
'not %s') % l)
'not %s') % lbl)
if k not in labels:
labels[k] = v
else: