From fcaccb600b76d189de0a373f954567ec7babc2d9 Mon Sep 17 00:00:00 2001 From: Dan Buch Date: Fri, 15 Jul 2016 08:45:46 -0400 Subject: [PATCH 1/2] Run flake8 checks on generate-flag-types --- .travis.yml | 2 +- generate-flag-types | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.travis.yml b/.travis.yml index d12ddbb..13e791c 100644 --- a/.travis.yml +++ b/.travis.yml @@ -38,7 +38,7 @@ before_script: - ln -sv ${TRAVIS_BUILD_DIR} ${GOPATH%%:*}/src/gopkg.in/urfave/cli.v2 script: -- flake8 runtests cli-v1-to-v2 +- flake8 runtests cli-v1-to-v2 generate-flag-types - ./runtests gen - ./runtests vet - ./runtests test diff --git a/generate-flag-types b/generate-flag-types index 28069b3..2ee0485 100755 --- a/generate-flag-types +++ b/generate-flag-types @@ -37,8 +37,8 @@ The meaning of each field is as follows: name (string) - The type "name", which will be suffixed with `Flag` when generating the type definition for `cli` and the wrapper type for `altsrc` - type (string) - The type that the generated `Flag` type for `cli` - is expected to "contain" as its `.Value` member + type (string) - The type that the generated `Flag` type for + `cli` is expected to "contain" as its `.Value` member value (bool) - Should the generated `cli` type have a `Value` member? dest (bool) - Should the generated `cli` type support a From a51cb870fbf3c34897dc080a848e60da3ee50ab0 Mon Sep 17 00:00:00 2001 From: Dan Buch Date: Fri, 15 Jul 2016 21:49:55 -0400 Subject: [PATCH 2/2] Wrap a long line for flake8 --- generate-flag-types | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/generate-flag-types b/generate-flag-types index 2ee0485..4ac7c04 100755 --- a/generate-flag-types +++ b/generate-flag-types @@ -38,7 +38,8 @@ The meaning of each field is as follows: `Flag` when generating the type definition for `cli` and the wrapper type for `altsrc` type (string) - The type that the generated `Flag` type for - `cli` is expected to "contain" as its `.Value` member + `cli` is expected to "contain" as its `.Value` + member value (bool) - Should the generated `cli` type have a `Value` member? dest (bool) - Should the generated `cli` type support a