Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

zsh completion gives errors #426

Closed
drnic opened this issue Oct 1, 2019 · 4 comments · Fixed by #439 or #443
Closed

zsh completion gives errors #426

drnic opened this issue Oct 1, 2019 · 4 comments · Fixed by #439 or #443
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@drnic
Copy link

drnic commented Oct 1, 2019

$ uname -a
Darwin starkair.lan 18.7.0 Darwin Kernel Version 18.7.0: Tue Aug 20 16:57:14 PDT 2019; root:xnu-4903.271.2~2/RELEASE_X86_64 x86_64

$ brew list zsh
/usr/local/Cellar/zsh/5.7.1/bin/zsh
...

$ source <(kn completion --zsh)
_arguments:comparguments:319: can only be called from completion function
_arguments:comparguments:319: can only be called from completion function

/cc @maximilien from #132 - any pro tips?

@drnic drnic added the kind/bug Categorizes issue or PR as related to a bug. label Oct 1, 2019
@rhuss
Copy link
Contributor

rhuss commented Oct 1, 2019

I can confirm the error.

My suggestion would be to either remove kn completion as it does not yet add many benefits (e.g. no options are included neither context-sensitive information like service names) or make a commitment to support it fully. At the moment it's less than half-baken.

@maximilien
Copy link
Contributor

I am OK with removing. I was hoping that the completion story in Cobra would get better out of the box. They made it easy to integrate but seems not so useful... It still does not work for me either 😞

@maximilien
Copy link
Contributor

Removed with #439

We can easily add again once/if the generated code from Cobra gets updated and actually works.

@navidshaikh
Copy link
Collaborator

We can easily add again once/if the generated code from Cobra gets updated and actually works.

Do we also have an issue filed against cobra for this ?

navidshaikh added a commit to navidshaikh/client that referenced this issue Oct 11, 2019
navidshaikh added a commit to navidshaikh/client that referenced this issue Oct 11, 2019
 Fixes knative#426

 and a couple of patches on top which aren't merged yet

 - spf13/cobra#884
 - spf13/cobra#899
navidshaikh added a commit to navidshaikh/client that referenced this issue Oct 23, 2019
 Fixes knative#426

 and a couple of patches on top which aren't merged yet

 - spf13/cobra#884
 - spf13/cobra#899
navidshaikh added a commit to navidshaikh/client that referenced this issue Oct 24, 2019
 Fixes knative#426

 - and a couple of patches on top which aren't merged yet
 	- spf13/cobra#884
	- spf13/cobra#899
 - also updates viper to 1.4.0
navidshaikh added a commit to navidshaikh/client that referenced this issue Oct 29, 2019
 Fixes knative#426

 - and a couple of patches on top which aren't merged yet
 	- spf13/cobra#884
	- spf13/cobra#899
 - also updates viper to 1.4.0
knative-prow-robot pushed a commit that referenced this issue Oct 30, 2019
* Updates spf13/cobra dep to v0.0.5

 Fixes #426

 - and a couple of patches on top which aren't merged yet
 	- spf13/cobra#884
	- spf13/cobra#899
 - also updates viper to 1.4.0

* Updates to go.sum

* Updates go.mod

 as a result of `go install golang.org/x/tools/cmd/goimports`
coryrc pushed a commit to coryrc/client that referenced this issue May 14, 2020
This changeset adds to support to perform coverage calculation based on
the TypeCoverage type. This change also includes support to provide
repos an ability to ignore certain fields based on their needs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
4 participants