Commit Graph

306 Commits

Author SHA1 Message Date
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
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
Sergey Romanov
f47f7b7e85 Fix panic if Valus in Int/StringSliceFlasg is missing 2015-06-01 01:50:23 +05:00
Fabian Ruff
65d50017d4 search context hierachy for global flags 2015-05-18 23:21:33 +02: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
Victor Vieux
7c041f5811 do not print 'AUTHOR(S):' is there is no author
Signed-off-by: Victor Vieux <victorvieux@gmail.com>
2015-05-03 18:01:26 -07:00
Jesse Szwedko
2e64a10881 Merge pull request #217 from codegangsta/readd-print-help-function
Readd printHelp function back
2015-05-03 17:52:12 -07:00
jszwedko
e842547421 Readd printHelp function back
But update signature to take a writer. This is a backwards incompatible
change for those overriding the HelpPrinter, but the hope is that this
feature is largely unused and the usage is easily updated.
2015-05-03 17:43:52 -07:00
Jesse Szwedko
30b361bf66 Merge pull request #211 from netbun/patch-1
changes to readme.md
2015-04-21 20:12:02 -07:00
My-khael Pierce
a66d4d2bec changes to readme.md 2015-04-12 14:24:35 -04:00
My-khael Pierce
e8af095ac4 changes to readme.md 2015-04-09 04:42:53 -04:00
Jesse Szwedko
9b2bd2b348 Merge pull request #205 from codegangsta/fix-aliases-readme
Names -> Aliases in README.md
2015-03-23 19:59:03 -07:00
jszwedko
6c6d93d451 Names -> Aliases in README.md 2015-03-23 07:55:18 -07:00
2bcd11f863 Merge pull request #193 from codegangsta/add-aliases
Add `Command.Aliases` and deprecate `Command.ShortName`
2015-03-16 21:49:33 -04:00
jszwedko
daad50e530 Turns out I didn't mean to add ExampleAppCommands at all 2015-03-16 18:48:06 -07:00
jszwedko
474646abed Didn't mean to remove ExampleAppSubcommand
Also specify struct field initializers
2015-03-16 18:04:57 -07:00
Jesse Szwedko
55e291873b Merge pull request #200 from smallfish/patch-1
Update help.go
2015-03-16 17:52:42 -07:00
陈小玉
84630daaf4 Update help.go
change template `AUTHOR(s)` tab ident to whitespace.
2015-03-14 01:12:37 +08:00
Jesse Szwedko
8ce64f19ff Merge pull request #187 from ravigadde/master
Added an API for number of flags set
2015-03-10 18:05:58 -07:00
Ravi Gadde
bcec9b08c7 Added an API and test case for number of flags set 2015-03-10 17:18:12 -07:00
Jesse Szwedko
012696c9b0 Merge pull request #196 from codegangsta/fix-deprecated-author-support
Fix support for deprecated author fields
2015-03-10 08:35:55 -07:00
jszwedko
b95607c608 Use named struct fields 2015-03-10 08:13:31 -07:00
jszwedko
7beac44ab1 Don't default authors 2015-03-10 08:03:05 -07:00
jszwedko
5f95a9e88b Fix support for deprecated author fields
Should add an author if either name or email is specified.
2015-03-10 07:59:59 -07:00
jszwedko
bf65971a6a Add Command.Aliases and deprecate Command.ShortName
`Aliases` will be more flexible while still allowing "ShortName"
behaviour via `Aliases`.
2015-03-09 21:24:57 -07:00
Jesse Szwedko
3e0905345c Merge pull request #186 from nrdufour/master
Fixing the issue with a command with subcommands not showing help message
2015-03-09 19:37:25 -07:00
Jesse Szwedko
842ff18343 Merge pull request #192 from codegangsta/add-multi-author-support
Add multi author support
2015-03-09 19:32:34 -07:00
jszwedko
01cdaa065a Merge remote-tracking branch 'origin/master' into add-multi-author-support
Conflicts:
	app.go
2015-03-09 19:20:56 -07:00
Nicolas Dufour
4be878bffc Fixing the issue with a command with subcommands not showing help message.
- the command name is "" and HasName was returning true for an empty
  ShortName.
- the Show method wasn't aware that command name was just "" and
  returned the first subcommand.
2015-02-23 08:32:31 -05:00
Harrison
c6592bb487 app, help: add backwards compatibility for Authors 2015-02-21 10:44:00 +11:00
jszwedko
50c77ecec0 Fix comments for .After field on Command and App 2015-02-20 16:23:36 -05:00
Jesse Szwedko
532d0bc2e3 Merge pull request #151 from imdario/master
After: companion action for Before
2015-02-20 16:20:12 -05:00
Jesse Szwedko
6086d7927e Merge pull request #177 from vishalsodani/master
Fix sentence
2015-01-30 19:12:59 -08:00
Harrison
05ecd63a95 app_test: prepare for PR 2015-01-31 10:05:45 +11:00
Harrison
3d7183307a app, help: add support for multiple authors 2015-01-31 10:04:52 +11:00
Vishal Sodani
2a9c617652 Fix sentence 2015-01-30 16:32:58 +05:30
Jesse Szwedko
e1712f3817 Merge pull request #170 from codegangsta/parse-flags-regardless
Use parsed context when running command as subcommnd
2015-01-27 21:16:17 -08:00
Dario Castañé
dca177c384 After handler brought up to speed.
As discussed in issue #151, I modified the implementation to be simpler.
I checked how to propagate err, being done with a scope trick. From our
deferred function we cannot return a new value, so we override the "default"
return value (err), declared on functions' declarations.

Check this StackOverflow for more info: http://stackoverflow.com/a/19934989
2015-01-10 00:51:06 +01:00
Dario Castañé
4ab12cd639 jszwedko's suggestion done 2015-01-10 00:44:37 +01:00
Dario Castañé
4e3a83b43e Merge branch 'master' of https://github.com/codegangsta/cli
Conflicts:
	app.go
	app_test.go
2015-01-10 00:35:11 +01:00
jszwedko
5159b98cbc Use parsed context when running command as subcommnd
Rather than passing the unparsed context (parent context) in whenever
there are no subcommands.
2015-01-09 14:46:29 -05:00
Jesse Szwedko
bf4a526f48 Merge pull request #157 from dajulia3/pull_request
stop flag parsing after terminator --
2015-01-09 14:04:33 -05:00
97112754e1 Merge pull request #169 from codegangsta/use-0-as-int-base
Use 0 as the base when parsing ints
2015-01-09 13:13:49 -05:00
jszwedko
44d40054fa Use 0 as the base when parsing ints
To be consistent with what the stdlib flag package does.
2015-01-09 13:10:42 -05:00