Commit Graph

1812 Commits

Author SHA1 Message Date
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
jhowarth
aba73cedac Copy the writer of the App to the subcommand App 2015-03-03 14:02:42 -08:00
jhowarth
a6482d2687 Merge remote-tracking branch 'upstream/master'
Conflicts:
	app.go
	command.go
	flag.go
2015-03-02 15:21:01 -08:00
Jesse Howarth
b5844af298 Merge pull request #2 from ivey/requiredFlags
Required flags
2015-03-02 14:53:57 -08:00
jhowarth
145da3210f don't require flags when the help flag is included 2015-03-02 12:06:42 -08:00
jhowarth
6023f370c1 dry error messages 2015-03-02 12:00:21 -08:00
jhowarth
e67e05f617 DRY error handling 2015-03-02 11:56:29 -08:00
jhowarth
cbd95292ac Remove debugging 2015-03-02 11:18:59 -08: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
Peter Smit
5308b4cd0f Allow commands to be hidden from help and autocomplete 2015-02-06 10:46:32 +02: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