Cleanups per flake8 and code review
This commit is contained in:
parent
659e1c1e0b
commit
e3a099f7c5
1
.gitignore
vendored
1
.gitignore
vendored
@ -1,2 +1 @@
|
|||||||
*.coverprofile
|
*.coverprofile
|
||||||
.extracted-examples
|
|
||||||
|
@ -2,8 +2,6 @@ language: go
|
|||||||
|
|
||||||
sudo: false
|
sudo: false
|
||||||
|
|
||||||
cache: pip
|
|
||||||
|
|
||||||
go:
|
go:
|
||||||
- 1.2.2
|
- 1.2.2
|
||||||
- 1.3.3
|
- 1.3.3
|
||||||
|
@ -171,13 +171,14 @@ def _migrate_v1_imports(source):
|
|||||||
def _migrate_new_exit_error(source):
|
def _migrate_new_exit_error(source):
|
||||||
return _EXIT_ERROR_RE.sub('cli.Exit', source)
|
return _EXIT_ERROR_RE.sub('cli.Exit', source)
|
||||||
|
|
||||||
|
|
||||||
@_migrator
|
@_migrator
|
||||||
def _migrate_bool_t_flag(source):
|
def _migrate_bool_t_flag(source):
|
||||||
return _BOOL_T_FLAG_RE.sub(_bool_t_flag_repl, source)
|
return _BOOL_T_FLAG_RE.sub(_bool_t_flag_repl, source)
|
||||||
|
|
||||||
|
|
||||||
def _bool_t_flag_repl(match):
|
def _bool_t_flag_repl(match):
|
||||||
return 'cli.BoolFlag{{Value: true, {}}}'.format(
|
return 'cli.BoolFlag{{Value: true,{}}}'.format(
|
||||||
match.groupdict()['args']
|
match.groupdict()['args']
|
||||||
)
|
)
|
||||||
|
|
||||||
@ -275,7 +276,7 @@ _MIGRATOR_TESTS = (
|
|||||||
\t\t\t\t&cli.StringFlag{
|
\t\t\t\t&cli.StringFlag{
|
||||||
\t\t\t\t\tName: "aha",
|
\t\t\t\t\tName: "aha",
|
||||||
\t\t\t\t},
|
\t\t\t\t},
|
||||||
\t\t\t\t&cli.BoolFlag{Value: true,
|
\t\t\t\t&cli.BoolFlag{Value: true,
|
||||||
\t\t\t\t\tName: "blurp",
|
\t\t\t\t\tName: "blurp",
|
||||||
\t\t\t\t},
|
\t\t\t\t},
|
||||||
\t\t\t}
|
\t\t\t}
|
||||||
|
Loading…
Reference in New Issue
Block a user