x/urfave-cli
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
696 Commits 2 Branches 0 Tags
bf7166a9d7df04137eb4fbd6704d4926c39c265d
Commit Graph
100 Commits
This Branch
This Branch
All Branches
Author SHA1 Message Date
Jesse Szwedko e7b1833f53 Dereference Flags if they are pointers 
When checking if environment variables are set.

We don't support pointer flags currently (though this is the default in
the `v2` branch), but this fixes #516
 2016-09-08 21:10:46 -07:00
Jesse Szwedko df95e0708f Manually set import in altsrc flag generation 
Otherwise `goimports` switches it to `github.com/urfave/cli`
 2016-09-03 14:22:45 -07:00
Jesse Szwedko 33bb4c1213 Merge pull request #511 from grubernaut/f-update-altsrc-import-v1 
Fix import paths in altsrc
 2016-08-28 13:23:14 -07:00
Jesse Szwedko a5ca09a934 fixup! remove the possiblity of end-user's seeing deprecation warnings 2016-08-21 14:06:59 -07:00
Jesse Szwedko 8e6aa34a12 remove the possiblity of end-user's seeing deprecation warnings 
Instead use deprecation pattern described in
https://blog.golang.org/godoc-documenting-go-code.

Fixes #507
 2016-08-21 13:51:55 -07:00
Jesse Szwedko 168c95418e Ensure that EnvVar struct field exists before interrogating it 
Otherwise you end up with `<invalid Value>` which, in practice, would
probably work, but this is cleaner.
 2016-07-31 20:11:16 -07:00
Jesse Szwedko 6c1f51aa95 Fix context.(Global)IsSet to respect environment variables 
This appeared to be the least messy approach to hack in support for
IsSet also checking environment variables to see if a particular
cli.Flag was set without making backwards incompatible changes to the
interface.

I intend to fix this more properly in v2, probably by adding another
method to the cli.Flag interface to push the responsibility down as it
occurred to me that it was really the `Flag`s themselves that offer
support for configuration via the environment as opposed to the
`context` or other supporting structures. This opens the door for the
anything implementing the `Flag` interface to have additional sources of
input while still supporting `context.IsSet`.
 2016-07-31 14:14:46 -07:00
Jesse Szwedko b616f60886 Note TOML support in README and CHANGELOG 2016-07-31 12:11:52 -07:00
Jesse Szwedko a71fcd8f23 Merge pull request #491 from ykanda/toml-support 
Add TOML config file loader
 2016-07-31 12:06:35 -07:00
Jesse Szwedko d9021faab6 Merge pull request #500 from npcode/readme-remove-v 
README: Remove unnecessary 'v' from version numbers
 2016-07-25 18:14:03 -07:00
Jesse Szwedko 65da20beab Correctly show help message if -h is provided to subcommand 
Currently, if an action is specified on a subcommand, it ignores the
`-h` and `--help` flags if passed to the subcommand, e.g.:

`foo bar -h`

would call the default action on `bar` rather than show the help
documentation.

Fixes #477
 2016-07-13 14:48:16 -07:00
Jesse Szwedko 73aa67b7a2 Merge pull request #466 from urfave/flag-context-gen 
Generate code for flag types and context accessors
 2016-07-13 14:07:04 -07:00
Jesse Szwedko 1efa31f08b Release v1.18.0 2016-06-27 22:30:56 -07:00
Jesse Szwedko 2ec9cfb7f9 Add missing fixes to CHANGELOG.md 2016-06-27 22:28:27 -07:00
Jesse Szwedko 4205e9c4ee Merge pull request #463 from urfave/help-command-categorization 
Help command categorization
 2016-06-22 09:55:33 -05:00
Jesse Szwedko 06a850dff5 Merge pull request #456 from urfave/uint-flags 
Add UintFlag, Uint64Flag types and supporting code
 2016-06-16 11:58:30 -04:00
Jesse Szwedko 7817f72540 Merge pull request #450 from urfave/flag-value-changelog-mention 
Mention #448 fix in change log
 2016-06-08 19:50:24 -07:00
Jesse Szwedko 01a3382359 Merge pull request #444 from urfave/more-v1-examples 
Add more v1 API examples
 2016-06-01 20:01:28 -07:00
Jesse Szwedko 2ae9042f5b Merge pull request #422 from urfave/update-references-to-codegangsta 
Update references to codegangsta to urfave (new org name)
 2016-05-22 14:46:02 -07:00
Jesse Szwedko 024b4c6240 Update references to codegangsta to urfave (new org name) 
Also add notice to README
 2016-05-22 14:42:23 -07:00
Jesse Szwedko bebbcf6d41 Merge pull request #417 from codegangsta/trim-go1.1-support 
Trim go1.1 support
 2016-05-21 11:46:57 -07:00
Jesse Szwedko 7f0ca9a349 Merge pull request #404 from codegangsta/runtests-argparse 
Use argparse in runtests
 2016-05-10 21:09:50 -07:00
Jesse Szwedko 92897b9bf6 Updating coverage badges 2016-05-10 20:42:38 -07:00
Jesse Szwedko 152e6ca11a Merge pull request #407 from codegangsta/coverage-breakout 
Separate badges for package coverage
 2016-05-10 20:39:12 -07:00
Jesse Szwedko f7b459e346 Merge pull request #406 from codegangsta/command-alias-help 
Fix command alias printing in help text
 2016-05-10 10:56:23 -07:00
Jesse Szwedko 9f09b40642 Merge pull request #400 from codegangsta/coverage 
Add a vet/test runner script with coverage on by default
 2016-05-09 22:16:32 -07:00
Jesse Szwedko 28eb7b2cc4 Added Hidden command support to CHANGELOG 2016-05-08 21:03:02 -07:00
Jesse Szwedko cf4a731612 Merge pull request #386 from codegangsta/psmit-hidden_command 
Merging #180 @psmit hidden command impl
 2016-05-08 21:01:54 -07:00
Jesse Szwedko 592f1d97e5 Exit non-zero if a unknown subcommand is given 
Currently it just prints the help message and exits 0.

We do this by modifying the helpCommand and helpSubcommand cli.Commands
to return an error if they are called with an unknown subcommand. This
propogates up to the app which exits with 3 and prints the error.

Thanks to @danslimmon for the initial approach!

Fixes #276
 2016-05-07 17:26:15 -07:00
Jesse Szwedko e3ace79a91 Add GlobalBoolT 
Fixes #206
 2016-05-07 16:11:26 -07:00
Jesse Szwedko d3a4d5467b Merge pull request #394 from mattfarina/travis-update 
Update travis config for Go versions
 2016-05-06 22:55:08 -07:00
Jesse Szwedko d919ffad32 Merge pull request #395 from mattfarina/goreportcard 
Adding Go Report Card badge
 2016-05-06 22:54:45 -07:00
Jesse Szwedko 839f07bfe4 Merge pull request #381 from codegangsta/pluggable-flag-printer 
Allow for pluggable flag-level help text formatting
 2016-05-02 21:17:00 -07:00
Jesse Szwedko 4733699ce3 Update changelog with placeholder support 2016-04-25 19:47:42 -07:00
Jesse Szwedko 7528b17cc6 Merge pull request #357 from kevin-cantwell/parse-usage-placeholders 
Parses usage placeholders via back quotes. Resolves #333
 2016-04-25 19:45:59 -07:00
Jesse Szwedko 71f57d300d Merge pull request #349 from Felamande/master 
use filepath.Base instead of path.Base in Name & HelpName as default.
 2016-04-03 18:54:34 -07:00
Jesse Szwedko bc465beccc Merge pull request #346 from codegangsta/category_sort_2 
Add option to make categories with command, to display a more structured help
 2016-03-26 15:39:47 -07:00
Jesse Szwedko 9fec0fad02 Merge pull request #343 from TimeIncOSS/f-hide-version 
Never show version if HideVersion=true
 2016-03-21 23:42:55 -05:00
Jesse Szwedko a7936f1d21 Merge pull request #342 from korzonek/master 
codebeat badge
 2016-03-20 15:44:03 -05:00
Jesse Szwedko a7be4a3f19 Describe category behavior in README 2016-03-20 12:18:28 -07:00
Jesse Szwedko 042842b819 Remove CategorizedHelp from App and allow subcommands to have categories 
Just place all subcommands in categories, the default category will be
"" which will properly format the output (and group commands that have
no category).

Also allow subcommands to have categories.

Lastly, augment the test to check the output.
 2016-03-20 12:17:13 -07:00
Jesse Szwedko aca5b047ed Drop support for Go 1.0.3 
Incompatible with the new alternate input sources and I don't see
a compelling reason to continue its support.
 2016-03-06 20:12:29 -08:00
Jesse Szwedko 6bb6787f33 Merge pull request #306 from ChrisPRobinson/inputfilesupport 
Update to add yaml support
 2016-03-06 20:12:11 -08:00
Jesse Szwedko a2943485b1 Merge pull request #336 from muraty/master 
Add NArg method to context
 2016-02-27 08:56:50 -08:00
Jesse Szwedko 5db74198de Merge pull request #334 from djui/patch-1 
Fix semantic typo
 2016-02-17 19:24:44 -08:00
Jesse Szwedko 0ab42fd482 Merge pull request #332 from adamclerk/fix/spellingErrors 
Fixes spelling issues and import alphabetical issues
 2016-02-09 20:42:30 -08:00
Jesse Szwedko 8cea2901d4 Merge pull request #331 from blaubaer/windows-ci 
Added ci configuration for windows using AppVeyor
 2016-02-08 21:40:34 -08:00
Jesse Szwedko d5f087da9d Merge pull request #289 from KSubedi/master 
Added command to have a custom text on the USAGE section of help
 2016-02-06 13:12:10 -08:00
Jesse Szwedko 94d8ca740d Merge pull request #321 from blaubaer/master 
Improve help messages for empty strings and do os specific envVar handling on windows
 2016-02-06 13:06:44 -08:00
Jesse Szwedko 3565e5dfe6 Merge pull request #322 from blaubaer/custom-error-handling 
Introduce custom usage error handling
 2016-02-06 12:58:47 -08:00
Jesse Szwedko 0d23d77bee Merge pull request #329 from leonardyp/master 
optimization of some places
 2016-02-06 11:30:49 -08:00
Jesse Szwedko cf1f63a727 Merge pull request #323 from hpcloud/master 
Fixed mishandling of a "-"(dash)-argument causing reordering of cli n…
 2016-01-29 19:19:57 -08:00
Jesse Szwedko 114097b816 Merge pull request #326 from tcyrus/patch-1 
Update README.md
 2016-01-28 18:17:57 -08:00
Jesse Szwedko f9cc3001e0 Merge pull request #315 from blaubaer/master 
Improved Before/After handling for Commands
 2016-01-21 21:44:38 -08:00
Jesse Szwedko ca6f864fa3 Merge pull request #320 from technosophos/master 
Remove panic from help.
 2016-01-21 21:11:01 -08:00
Jesse Szwedko c31a797586 Merge pull request #311 from Jille/exportFlagNames 
Export cli.Flag.GetName (previously cli.Flag.getName)
 2015-12-21 13:40:36 -05:00
Jesse Szwedko 8eb7738f18 Merge pull request #312 from ysh7/master 
use path.Base in Name & HelpName as default values
 2015-12-21 13:39:19 -05:00
Jesse Szwedko b5232bb293 Merge pull request #309 from dedalusj/check-completion-order 
Run check completion before error checking
 2015-12-12 19:52:18 -05:00
Jesse Szwedko a3c75bc94e Merge pull request #304 from osocurioso/silence-tests 
test: avoid output from "go test" when tests pass
 2015-12-12 17:25:10 -05:00
Jesse Szwedko 0302d3914d Merge pull request #283 from codegangsta/really-skip-flag-parsing 
Really skip flag parsing
 2015-11-20 13:56:42 -08:00
Jesse Szwedko c75c9d0182 Merge pull request #298 from codegangsta/use-correct-example-naming 
Use correct example naming and test on Go tip
 2015-11-15 13:47:47 -08:00
Jesse Szwedko 0f218fffa5 Remave cli_test.go 
Examples are redundant with the ones in app_test.go
 2015-11-15 13:34:53 -08:00
Jesse Szwedko 631930114f Also test on Go tip 
But allow it to fail
 2015-11-15 13:14:18 -08:00
Jesse Szwedko 4fc241fb17 Update examples to use correct naming convention 
See https://golang.org/pkg/testing/#hdr-Examples
 2015-11-15 13:14:11 -08:00
Jesse Szwedko 49bed4ccde Merge pull request #295 from rosenhouse/update-travis-versions 
Update golang versions used by Travis CI
 2015-11-15 13:05:45 -08:00
Jesse Szwedko 93b4ad6d1f Merge pull request #297 from ston1th/master 
Added destination scan support for flags
 2015-11-15 12:34:56 -08:00
Jesse Szwedko 8cd49b108c Update TestCommandIgnoreFlagsIncludingFirstArgument to test for arguments 2015-10-29 20:16:35 -07:00
Jesse Szwedko 6191d931b7 When skipping flag parsing, still parse into arguments 
Fool the FlagSet into thinking that all arguments are actually
arguments rather than attempting to parse them as flags.
 2015-10-29 20:16:35 -07:00
Jesse Szwedko bc3cb33cef Actually skip parsing of flags if SkipFlagParsing is set 
Previous just skipped if firstFlagIndex was > -1
 2015-10-29 20:16:35 -07:00
Jesse Szwedko 70e3fa51eb Merge pull request #292 from Jille/remove-unused-var 
bash_autocomplete: Remove unused local variable prev
 2015-10-29 20:11:27 -07:00
Jesse Szwedko f445c89440 Merge pull request #284 from rbjorklin/readme-update 
Added coverage and reference logos to README.md
 2015-10-21 22:00:38 -07:00
Jesse Szwedko cffab77ecb Merge pull request #286 from mktmpio/dedup-help-and-version 
only display app version and help message once
 2015-10-18 17:28:50 -07:00
Jesse Szwedko aba9469083 Merge pull request #282 from cactus/master 
make help and version parsing use actual specified values
 2015-10-17 11:54:49 -07:00
Jesse Szwedko 96f1fedb28 Merge pull request #281 from thiderman/patch-1 
Remove 'bitchass' from README
 2015-10-06 19:46:01 -07:00
Jesse Szwedko 9039757268 Merge pull request #267 from tristanz/master 
Add ArgsUsage field to document the use of arguments
 2015-10-06 19:45:04 -07:00
Jesse Szwedko 88df06b5e3 Merge pull request #279 from nodirt/version 
Honor HideHelp and HideVersion
 2015-10-05 20:26:32 -07:00
Jesse Szwedko f35cf683a8 Merge pull request #280 from gummiboll/fix-for-testcommandignoreflags 
Added a flag for TestCommandIgnoreFlags
 2015-10-05 20:05:46 -07:00
Jesse Szwedko a65b733b30 Merge pull request #275 from langri-sha/master 
Improve formatting
 2015-09-08 18:48:53 -07:00
Jesse Szwedko eaad6fe6bc Merge pull request #274 from klueska/bash-completion-default-prog 
Set default PROG in bash_autocomplete
 2015-09-05 14:07:14 -07:00
Jesse Szwedko 142e6cd241 Merge pull request #264 from wardn/master 
Improve vendoring options by removing self-referential imports in tests.
 2015-08-11 21:22:59 -07:00
Jesse Szwedko bca61c476e Merge pull request #246 from cpuguy83/propigate_cmd_parent_to_app_setup 
Sets a subcommand's parent cmd
 2015-07-11 14:54:04 -07:00
Jesse Szwedko f9982619cc Merge pull request #251 from MartyMacGyver/more-cleanups 
More cleanups
 2015-06-30 20:15:21 -07:00
Jesse Szwedko b6d983e02c Merge pull request #253 from databus23/fix-isssue-252 
Fix global flags processing on top level
 2015-06-29 21:01:06 -07:00
Jesse Szwedko ad480243a8 Merge pull request #245 from cpuguy83/get_parent_ctx 
Add func to get context parent
 2015-06-20 11:34:02 -07:00
Jesse Szwedko 7ad88c2740 Merge pull request #238 from polds/copyright 
Add the ability to add a copyright
 2015-06-16 19:39:24 -07:00
Jesse Szwedko 8cbee4b719 Merge pull request #244 from wallclockbuilder/patch-1 
Fix tiny typo.
 2015-06-11 17:23:08 -05:00
Jesse Szwedko 11ce503420 Merge pull request #241 from codegangsta/travis-update 
Update Travis config to run against more versions on faster CI
 2015-06-11 00:35:07 -05:00
Jesse Szwedko 616b730509 Merge pull request #233 from codegangsta/add-flag-docs 
Add godoc comments to flag structs
 2015-06-05 09:38:13 -07:00
Jesse Szwedko 005b120d20 Add godoc comments to flag structs 2015-06-02 20:51:09 -07:00
Jesse Szwedko 55b8924c2b Merge pull request #230 from codegangsta/do-not-swallow-after-error 
Aggregate errors from Before/After
 2015-06-02 20:29:44 -07:00
Jesse Szwedko 2272dad83e Version and help check should look for local flags too 
Now that Global looks up the chain of contexts, the top level should
access the flags without the prefix (i.e. Bool rather than GlobalBool).
 2015-06-02 20:16:44 -07:00
Jesse Szwedko 854c52192c Merge pull request #227 from databus23/global-flags-from-all-parents 
Collect global flags from the full command hierarchy
 2015-06-01 21:25:30 -07:00
Jesse Szwedko 7ed7a51f86 Aggregate errors from Before/After 
Previously `After` would overwrite any error from `Before`.
 2015-06-01 21:14:53 -07:00
Jesse Szwedko 5d58d5b313 Merge pull request #229 from saromanov/fix-missing-value 
Fix panic if Valus in Int/StringSliceFlasg is missing
 2015-06-01 20:39:11 -07:00
jszwedko 942282e931 Add instructions for distributing bash completion 
[ci skip]
 2015-05-10 11:45:54 -07:00
jszwedko a889873af5 Set additional information on command's 'app' 
For completeness
 2015-05-03 18:42:21 -07:00
Jesse Szwedko 4ee3fafcf8 Merge pull request #218 from codegangsta/add-another-help-test 
Set writer when running command as app
 2015-05-03 18:41:04 -07:00
jszwedko b8104e5da7 Set writer when running command as app 
Also add test from https://github.com/codegangsta/cli/pull/202 with
slight modifications.
 2015-05-03 18:37:51 -07:00
jszwedko f952f5ac6f Rename TestShowAppHelp to be more verbose 
Follow golang convention
 2015-05-03 18:04:45 -07:00
jszwedko 96ad929784 Add test for inclusion of 'AUTHORS' in App help 2015-05-03 18:02:03 -07:00