| Age | Commit message (Collapse) | Author |
|---|
|
Signed-off-by: Oren Shomron <[email protected]>
|
|
They have to explicitly unhide a deprecated flag to keep the current behavior.
|
|
* add ability to ignore unknown flags
* add testcases
* add 2 more patterns to the testcase
* handle --unknownflag=val arg and -u=val arg scenario
* add ParseErrorsWhiteList to extend error handling during parsing
|
|
|
|
* add int16 flag
copied int32.go to int16.go and s/32/16
added a case to testParse in flag_test.go
* test the set value of int16 flag
|
|
|
|
|
|
Related to https://github.com/spf13/cobra/issues/521
|
|
|
|
|
|
|
|
|
|
flag_test.go:338: result of fmt.Errorf call not used
flag_test.go:377: no formatting directive in Errorf call
|
|
Fix #120
|
|
Fixes https://github.com/spf13/cobra/issues/316
|
|
Clean up code
|
|
We basically want to allow callers to use our command line parser. There
is a user who wants to log the command line, but they don't want to log
"sensitive" flags. This allows that user to parse the command line and
get each flag. They can use information in the flag to write their own
printer.
|
|
|
|
Fallback to the original default detection logic if we have an unknown
type.
Fixes #91
|
|
|
|
In Go 1.7 the default value for a time.Duration will change from "0" to
"0s". This commit replaces the simplistic `isZeroValue` with a more
intelligent implementation that checks for zero values based
upon the specific type of the flag.
|
|
Compare to 1.4, the main difference in formatting is
the placement of default values. Moreover, UnquoteUsage()
is now added (and exported, for full API compatibility
with the standard flag package), so backtick words in
usage messages can be used to set the placeholder name.
Compared to the standard flag package, this patch
always prints usage in one-line, with automatic
alignment, because I feel that the 1.4 output is very
confusing when modified to include also dash-dash flags.
|
|
This allows a program to know what args came before the -- and what args
came after.
|
|
removing some c/p left-overs from the MarkPrivate function, and updating the comment slightly
changed field from Private to Hidden. Added documentation w/example to README.md. Added testing to confirm hidden flags
updating test message
|
|
|
|
This function allows one to create a different FlagSets and then easily
merge them together.
|
|
|
|
|
|
A lot easier than having to do it in the user over and over and over.
|
|
|
|
|
|
Add new FlagSet.Get{Int,String,...} accessor functions
|
|
|
|
If I declared a bool flag named "hello" I can now call
b, err := f.GetBool("hello")
And b will hold the value of the flag
We can see this is already done in
https://github.com/codegangsta/cli/blob/bcec9b08c7e5564f7512ad7e7b03778fe1923116/context.go
If people use the codegangsta/cli
Other projects have done it themselves using pflags (what inspired this
patch)
https://github.com/GoogleCloudPlatform/kubernetes/blob/cd817aebd848facda29e0befbbd6e31bf22402e6/pkg/kubectl/cmd/util/helpers.go#L176
Lets just do it ourselves...
|
|
|
|
[close #23]
|
|
|
|
Currently a shortname bool variable b cannot use the -b=true -b=false
form because the -b is assumed to mean true and the = format is never
noticed.
|
|
Conflicts:
bool_test.go
flag.go
flag_test.go
|
|
This can be used however you want. My test is for flag aliases
|
|
They will not show up in usage or help, but they will still work.
The usage message will print on os.Stderr any time the flag is set.
|
|
Add support to short form with equal sign: -p=value
|
|
|
|
This allows the FlagSet to define equivalent word-separators. Given
this, one can make - and _ equivalent, so --foo-bar and --foo_bar both
work.
|
|
|
|
Fix handling of -- termination. Previously, if you ran
$command -flag1 -etc -- some args after
the command's args would be doubled (some args after some args after).
Additionally, if there were items after the -- termination that looked
like flags, the parser would throw an error. It's better to terminate
parsing when -- is seen, because that usually indicates the caller wants
to pass the remainder of the arguments (including flags) as-is to
something else.
|
|
This means we can avoid putting the repository name into the source
files, so people who have their own forks don't have to make changes.
|
|
This was broken by commit 463bdc838f2b35 ("Adding Type() method to the values.") which added the method to the interface but did not update the custom types in unit tests.
Signed-off-by: Filipe Brandenburger <[email protected]>
|
|
|
|
|
