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

Make bash completions work on a Mac #13570

Merged
2 commits merged into from
Sep 4, 2015
Merged

Conversation

eparis
Copy link
Contributor

@eparis eparis commented Sep 3, 2015

No description provided.

@eparis
Copy link
Contributor Author

eparis commented Sep 3, 2015

No, the help flags are not actually disappearing because of this update to cobra, just their appearance in the docs....

@k8s-github-robot k8s-github-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 3, 2015
@k8s-github-robot
Copy link

Labelling this PR as size/L

@eparis eparis force-pushed the completions-on-mac branch 2 times, most recently from a22ce07 to 8a45deb Compare September 4, 2015 14:16
@k8s-bot
Copy link

k8s-bot commented Sep 4, 2015

GCE e2e build/test failed for commit 8a45debba2f24a4be164ca47a9f6cf02c914bc98.

@eparis
Copy link
Contributor Author

eparis commented Sep 4, 2015

ok to test

@ghost
Copy link

ghost commented Sep 4, 2015

Thanks @eparis. Could you split the spf13 upgrade into a separate PR please?

@ghost ghost assigned janetkuo Sep 4, 2015
@eparis
Copy link
Contributor Author

eparis commented Sep 4, 2015

Not really. All this commit does it update spf13/cobra and regen docs. Those need to be semi-atomic.

@ghost
Copy link

ghost commented Sep 4, 2015

Fair enough. Needs a rebase and merge conflict resolution. Then LGTM.

@eparis
Copy link
Contributor Author

eparis commented Sep 4, 2015

updated. Thanks!

@k8s-bot
Copy link

k8s-bot commented Sep 4, 2015

GCE e2e build/test passed for commit bf7646b.

ghost pushed a commit that referenced this pull request Sep 4, 2015
Make bash completions work on a Mac
@ghost ghost merged commit 20b90ee into kubernetes:master Sep 4, 2015
@eparis eparis deleted the completions-on-mac branch September 16, 2015 04:09
@@ -109,7 +108,7 @@ $ kubectl scale --replicas=5 rc/foo rc/bar

* [kubectl](kubectl.md) - kubectl controls the Kubernetes cluster manager

###### Auto generated by spf13/cobra at 2015-08-21 17:18:05.165785015 +0000 UTC
###### Auto generated by spf13/cobra at 2015-09-03 21:06:22.471116954 +0000 UTC

<!-- BEGIN MUNGE: GENERATED_ANALYTICS -->
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

O

@@ -83,7 +82,7 @@ kubectl version

* [kubectl](kubectl.md) - kubectl controls the Kubernetes cluster manager

###### Auto generated by spf13/cobra at 2015-08-21 17:18:05.1688832 +0000 UTC
###### Auto generated by spf13/cobra at 2015-09-03 21:06:22.476464324 +0000 UTC

<!-- BEGIN MUNGE: GENERATED_ANALYTICS -->
[![Analytics](https://kubernetes-site.appspot.com/UA-36037335-10/GitHub/docs/user-guide/kubectl/kubectl_version.md?pixel)]()

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is this ga analytics beacon tracker & badge ?!?? bad-ass. what (if any) open-source project does it stem from?

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ui size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants