diff options
author | David Pursehouse <david.pursehouse@sonymobile.com> | 2012-10-11 16:44:48 +0900 |
---|---|---|
committer | David Pursehouse <david.pursehouse@sonymobile.com> | 2012-10-22 12:30:14 +0900 |
commit | 5c6eeac8f0350fd6b14cf226ffcff655f1dd9582 (patch) | |
tree | 3225695b9d2a97342a49127717ea5e2bc5935a63 /subcmds/abandon.py | |
parent | e98607248eec2b149d84efe944c12cbef419b82e (diff) | |
download | git-repo-5c6eeac8f0350fd6b14cf226ffcff655f1dd9582.tar.gz |
More coding style cleanup
Fixing more issues found with pylint. Some that were supposed to
have been fixed in the previous sweep (Ie0db839e) but were missed:
C0321: More than one statement on a single line
W0622: Redefining built-in 'name'
And some more:
W0631: Using possibly undefined loop variable 'name'
W0223: Method 'name' is abstract in class 'name' but is not overridden
W0231: __init__ method from base class 'name' is not called
Change-Id: Ie119183708609d6279e973057a385fde864230c3
Diffstat (limited to 'subcmds/abandon.py')
-rw-r--r-- | subcmds/abandon.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/subcmds/abandon.py b/subcmds/abandon.py index 42abb2ff..e17ab2b6 100644 --- a/subcmds/abandon.py +++ b/subcmds/abandon.py | |||
@@ -42,10 +42,10 @@ It is equivalent to "git branch -D <branchname>". | |||
42 | nb = args[0] | 42 | nb = args[0] |
43 | err = [] | 43 | err = [] |
44 | success = [] | 44 | success = [] |
45 | all = self.GetProjects(args[1:]) | 45 | all_projects = self.GetProjects(args[1:]) |
46 | 46 | ||
47 | pm = Progress('Abandon %s' % nb, len(all)) | 47 | pm = Progress('Abandon %s' % nb, len(all_projects)) |
48 | for project in all: | 48 | for project in all_projects: |
49 | pm.update() | 49 | pm.update() |
50 | 50 | ||
51 | status = project.AbandonBranch(nb) | 51 | status = project.AbandonBranch(nb) |