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

Added a mention of 'oc get' to 'oc get --help' #10775

Merged
merged 1 commit into from
Sep 20, 2016
Merged

Added a mention of 'oc get' to 'oc get --help' #10775

merged 1 commit into from
Sep 20, 2016

Conversation

oatmealraisin
Copy link

Fixes #9326

@juanvallejo @fabianofranz @deads2k @liggitt ptal?

New output:

 $ oc get --help
Display one or many resources

Possible resources include builds, buildConfigs, services, pods, etc. To see a
list of common resources, use 'oc get'. Some resources may omit
advanced details that you can see with '-o wide'.  If you want an even more
detailed view, use 'oc describe'.

@juanvallejo
Copy link
Contributor

@fabianofranz LGTM

@liggitt
Copy link
Contributor

liggitt commented Sep 1, 2016

fails man page tests, also not a blocker, so post-1.3

@liggitt liggitt added this to the 1.3.1 milestone Sep 1, 2016
@oatmealraisin
Copy link
Author

Would anyone object to adding the same verbage to oc describe? Same situation there. @juanvallejo @liggitt

@fabianofranz
Copy link
Member

LGTM, just need to add generated docs from hack/update-generated-docs.sh. Merge will happen after the 1.3 release.

@smarterclayton
Copy link
Contributor

Needs updated docs still

@smarterclayton smarterclayton modified the milestones: 1.4.0, 1.3.1 Sep 19, 2016
@bparees
Copy link
Contributor

bparees commented Sep 20, 2016

@oatmealraisin bump, needs docs gen.

@bparees
Copy link
Contributor

bparees commented Sep 20, 2016

hm. actually looks like they are up to date now.
[merge]

@bparees
Copy link
Contributor

bparees commented Sep 20, 2016

flake #11008
[merge]

1 similar comment
@bparees
Copy link
Contributor

bparees commented Sep 20, 2016

flake #11008
[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 8394d29

@openshift-bot
Copy link
Contributor

openshift-bot commented Sep 20, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/9028/) (Image: devenv-rhel7_5042)

@openshift-bot openshift-bot merged commit b39e67f into openshift:master Sep 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants