glenux-archive/spf13--cobra
1
0
Fork 0
mirror of https://github.com/spf13/cobra synced 2024-11-24 22:57:12 +00:00
Code Issues Projects Releases Packages Wiki Activity
368 commits 4 branches 24 tags 32 MiB
Commit graph

368 commits

This branch
This branch
All branches
Author SHA1 Message Date
John Schnake
853cb4d97f Issue #267: Ensure hidden flags/subcommands do not change help 
If a command has one flag which is hidden, it should not, for
instance, show the `Flags: ` heading.  Likewise there are other
items in the help template which should respect hidden/deprecated
state.
 2016-04-13 13:27:05 -05:00
Eric Paris
4c05eb1145 Merge pull request #265 from sttts/sttts-shellcheck 
Run shellcheck on bash completion
 2016-04-03 09:08:35 -05:00
Dr. Stefan Schimanski
defeccc04d Run shellcheck on bash completion 2016-04-02 23:16:12 +02:00
Eric Paris
60267ae24d Merge pull request #257 from sttts/sttts-noun-aliases 
Add noun aliases
 2016-04-02 09:29:34 -05:00
Dr. Stefan Schimanski
1a0a490d76 Remove plural handling and add aliases for nouns in completion 2016-04-02 12:39:19 +02:00
Eric Paris
2ccf9e982a Merge pull request #264 from kkirsche/patch-1 
Update template per to match previous pr with golint fix
 2016-04-01 21:14:51 -05:00
Kevin Kirsche
3e71c61197 Update init per golint 
X-Ref: #258
 2016-04-01 21:29:12 -04:00
Eric Paris
b0d571e7d5 Merge pull request #260 from ianwalter/code-cleanup 
Fixing golint warnings
 2016-03-31 09:32:10 -05:00
Ian Walter
a0bd6c17b3 Fixing golint warnings 
* Moving final return outside of if-else
* Removing type declarations that Go can infer from values
* Cleaning up some existing comments
* Changing snake_case variables to camelCase
 2016-03-31 09:53:34 -04:00
Eric Paris
ee150b4fa2 Merge pull request #258 from kkirsche/patch-1 
Update root.go to fix golint bug with comment
 2016-03-30 16:59:46 -05:00
Kevin Kirsche
f643f707c9 Update root.go to fix golint bug with comment 
Fix the name in the comment to fix a golint bug by not having the variable name begin the comment.
 2016-03-30 17:50:52 -04:00
Eric Paris
c678ff029e Merge pull request #256 from sttts/sttts-fix-bash-escaping 
Fix escaping of % in bash code strings
 2016-03-22 12:10:42 -05:00
Dr. Stefan Schimanski
a23d903a12 Fix escaping of % in bash code strings 2016-03-22 17:44:59 +01:00
Eric Paris
966e6048eb Merge pull request #230 from garthk/fix-man-see-also 
Ensure SEE ALSO list has no leading comma.
 2016-03-21 19:23:55 -05:00
Eric Paris
57021c6b4d Merge pull request #244 from apriendeau/silence-errors-readme 
updates readme to Explain SilenceErrors and SilenceUsage #225
 2016-03-21 19:22:44 -05:00
Eric Paris
d6b481239e Merge pull request #255 from sttts/sttts-bash-completion-flag-equal 
Advanced custom bash completion of flags
 2016-03-21 19:22:24 -05:00
Dr. Stefan Schimanski
ea9a77d312 Add support for plural nouns in bash completion 2016-03-21 08:05:54 +01:00
Dr. Stefan Schimanski
8092588fb8 Add custom bash flag completion 2016-03-21 08:05:54 +01:00
Dr. Stefan Schimanski
7d556a0974 Enable bash completion after --flag= or --flag=abc 2016-03-21 08:05:49 +01:00
Eric Paris
1bacefc9a2 Merge pull request #254 from fabianofranz/optional_options_at_the_end_of_usage 
Must only add "[flags]" to the end of usage if not yet present
 2016-03-15 16:50:08 -05:00
Fabiano Franz
16b7ba55fb Must only add "[flags]" to the end of usage if not yet present 2016-03-15 18:22:20 -03:00
Austin Riendeau
e21e5f0b09 updates readme to Explain SilenceErrors and SilenceUsage #225 2016-02-18 13:57:07 -07:00
Anthony Fok
65a708cee0 Allow bash-completion to work with full-path executable 2016-02-08 17:06:27 -05:00
Jonathan Reyna
1ef0913976 Use nil instead of len for arg check. 
This honors the SetArgs() call
 2016-02-08 17:04:25 -05:00
Steve Francia
bbdcadf158 Add Swarm to Cobra user list 2016-02-08 17:02:59 -05:00
Eric Paris
04d02e3184 Merge pull request #241 from bogem/typo 
Fix typo in README
 2016-02-06 12:39:43 -05:00
bogem
64cc91ac9b Fix typo in README 2016-02-06 17:38:32 +01:00
Anthony Fok
801364aa15 Skip "bash: declare: -A: invalid option" error gracefully 
The use of "declare -A flaghash" (associative array) was introduced
in PR #205, which works perfectly for Bash 4.x, but OS X insists on
shipping a very outdated Bash 3.2.x.

This patch hides the "bash: declare: -A: invalid option" error message
and allows the bash completion script to continue gracefully on
OS X, albeit without the benefit of the new feature in PR #205.

Fixes #240
 2016-02-05 13:04:36 +08:00
Garth Kidd
57f473263e Add explicit doc.GenMan SEE ALSO test covering #229 2016-01-21 12:12:26 +11:00
Garth Kidd
cb8496d6b2 Ensure SEE ALSO list has no leading comma, fixing #229 2016-01-21 12:11:56 +11:00
Eric Paris
8e91712f17 Merge pull request #228 from garthk/fix-doc-generation 
Fix doc generation and its documentation
 2016-01-16 22:09:38 -05:00
Garth Kidd
356750645f Update doc generation examples. Fixes #227. 2016-01-17 11:29:42 +11:00
Garth Kidd
8e6aca4182 Use filepath.Join, not string math. Fixes #226. 2016-01-17 11:25:20 +11:00
Eric Paris
9c93009019 Merge pull request #223 from mattnenterprise/doc-link-fixes 
Fix links to markdown and man page docs
 2016-01-13 22:06:19 -05:00
Matt McCoy
d12e2412b6 Fix links to markdown and man page docs 2016-01-13 21:40:50 -05:00
Anthony Fok
97f831a4fd Change first line of generated bash-completion file 
to conform with existing bash-completion snippets and
to suppress Lintian "script-not-executable".
 2016-01-11 15:37:32 -07:00
Gyu-Ho Lee
2a426b5c59 README: add etcd 2016-01-06 16:54:07 -05:00
Eric Paris
a3a9f345ca Merge pull request #218 from mvdan/error-checking 
Error checking for doc generation
 2016-01-06 09:57:15 -05:00
Daniel Martí
5df1341f93 Treat write errors in man doc generation 
Just like the last commit, but now for manpages.

genMan still works with a buffer and returns []byte instead of working
directly with an io.Writer. This is because, in turn, md2man takes byte
slices instead of readers and writers.

Wrapping genMan around a writer is unnecessary especially since it's not
an exported function, and also because we'd still need a buffer to get
the output bytes.
 2016-01-06 12:25:27 +01:00
Daniel Martí
eb5040e69e Treat write errors in markdown doc generation 
This code was already using io.Writer, but was completely ignoring write
errors.

The most worrying part is how GenMarkdownTreeCustom used an unnecessary
buffer to then dump all of its contents on a file, and instead of
returning an error on file creation/writing, it would just exit the
entire program.
 2016-01-06 12:21:04 +01:00
Daniel Martí
ea06b29c10 Simplify GenMarkdownTreeCustom signature 2016-01-06 11:59:08 +01:00
Daniel Martí
5fa1003a36 Remove unused outs global var 2016-01-06 11:49:16 +01:00
Eric Paris
c82a0ceef8 Merge pull request #215 from mvdan/interfacer 
Replace *bytes.Buffer with io.Writer
 2016-01-05 16:50:52 -05:00
Daniel Martí
e0479ff7a7 Add error handling to GenBashCompletion 
Since the switch from *bytes.Buffer to io.Writer, errors can no longer
be ignored. Also makes the reuse of GenBashCompletion in
GenBashCompletionFile without a buffer treat errors properly again.
 2016-01-05 21:31:19 +01:00
Daniel Martí
a5ff2f28ab Reuse GenBashCompletion in GenBashCompletionFile 2016-01-05 21:30:42 +01:00
Daniel Martí
463be1428d Replace *bytes.Buffer with io.Writer 
Also adds support for generating bash completions on writers other than
just buffers.

Found via github.com/mvdan/interfacer.
 2016-01-05 21:30:42 +01:00
Tamer TAS
c6399cd398 Use readline's default filename completion if completion fails 
If the `compspec` generates no matches it uses bash filename completion instead.
 2016-01-02 08:11:35 +02:00
Frank Schroeder
b3f29e98e6 Issue #195: Move doc generation into separate pkg 
* Move man_docs and md_docs into new doc pkg
* Replace *bytes.Buffer with io.Writer
* Replace c == cmd.helpCommand with c.IsHelpCommand()
* Remove redundant len(children) == 0 check in HasSeeAlso
* Duplicate test setup for doc generation
 2015-12-24 09:01:59 -05:00
Steve Domino
f3aca0a8e7 adding nanobox/nanopack to list of cobra users 2015-12-24 08:54:09 -05:00
Kenjiro Nakayama
60b8213eba Add flaghash to bash_completion to keep flag value 2015-12-24 08:53:30 -05:00