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 | ||
