diff options
author | David Pursehouse <dpursehouse@collab.net> | 2020-02-12 15:20:19 +0900 |
---|---|---|
committer | David Pursehouse <dpursehouse@collab.net> | 2020-02-12 06:36:40 +0000 |
commit | 819827a42ddb364f98c3a1a7eae2536dc54bc4cc (patch) | |
tree | fe6bdca5ff7e44d53595a6da76d2b56ea659eee1 /subcmds/grep.py | |
parent | abdf7500612f1d115863ba8f026ddbea1e5a1f28 (diff) | |
download | git-repo-819827a42ddb364f98c3a1a7eae2536dc54bc4cc.tar.gz |
Fix blank line issues reported by flake8
- E301 expected 1 blank line
- E302 expected 2 blank lines
- E303 too many blank lines
- E305 expected 2 blank lines after class or function definition
- E306 expected 1 blank line before a nested definition
Fixed automatically with autopep8:
git ls-files | grep py$ | xargs autopep8 --in-place \
--select E301,E302,E303,E305,E306
Manually fix issues in project.py caused by misuse of block comments.
Change-Id: Iee840fcaff48aae504ddac9c3e76d2acd484f6a9
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/254599
Reviewed-by: Mike Frysinger <vapier@google.com>
Tested-by: David Pursehouse <dpursehouse@collab.net>
Diffstat (limited to 'subcmds/grep.py')
-rw-r--r-- | subcmds/grep.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/subcmds/grep.py b/subcmds/grep.py index 4dd85d57..13069286 100644 --- a/subcmds/grep.py +++ b/subcmds/grep.py | |||
@@ -23,12 +23,14 @@ from command import PagedCommand | |||
23 | from error import GitError | 23 | from error import GitError |
24 | from git_command import git_require, GitCommand | 24 | from git_command import git_require, GitCommand |
25 | 25 | ||
26 | |||
26 | class GrepColoring(Coloring): | 27 | class GrepColoring(Coloring): |
27 | def __init__(self, config): | 28 | def __init__(self, config): |
28 | Coloring.__init__(self, config, 'grep') | 29 | Coloring.__init__(self, config, 'grep') |
29 | self.project = self.printer('project', attr='bold') | 30 | self.project = self.printer('project', attr='bold') |
30 | self.fail = self.printer('fail', fg='red') | 31 | self.fail = self.printer('fail', fg='red') |
31 | 32 | ||
33 | |||
32 | class Grep(PagedCommand): | 34 | class Grep(PagedCommand): |
33 | common = True | 35 | common = True |
34 | helpSummary = "Print lines matching a pattern" | 36 | helpSummary = "Print lines matching a pattern" |
@@ -156,7 +158,6 @@ contain a line that matches both expressions: | |||
156 | action='callback', callback=carry, | 158 | action='callback', callback=carry, |
157 | help='Show only file names not containing matching lines') | 159 | help='Show only file names not containing matching lines') |
158 | 160 | ||
159 | |||
160 | def Execute(self, opt, args): | 161 | def Execute(self, opt, args): |
161 | out = GrepColoring(self.manifest.manifestProject.config) | 162 | out = GrepColoring(self.manifest.manifestProject.config) |
162 | 163 | ||