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

edit cf CLI dev guide README #293

Merged
merged 3 commits into from
Nov 12, 2014
Merged

edit cf CLI dev guide README #293

merged 3 commits into from
Nov 12, 2014

Conversation

jennjblack
Copy link

cf-docs team README edits for cf CLI 6.7.0

@cfdreddbot
Copy link

Hey jennjblack!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you've already signed the CLA.

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this. You can view the current status of your issue at: https://www.pivotaltracker.com/story/show/82323578.

a plugin.CliConnection. The plugin.CliConnection is a struct containing methods
for invoking cf cli commands. The second argument to Run(..) is a slice
containing the arguments passed from the `cf` process.
- A struct `plugin.CliConnection` that containins methods for invoking cf cli commands
Copy link
Contributor

Choose a reason for hiding this comment

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

Spelling mistake for 'contains'

@jennjblack
Copy link
Author

Fixed that typo and also fixed other formatting issues.

@jennjblack jennjblack closed this Nov 12, 2014
@jennjblack jennjblack reopened this Nov 12, 2014

## Installing Plugins
The `plugin.CliCommand([]args)` returns the output printed by the command
Copy link
Contributor

Choose a reason for hiding this comment

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

plugin.CliCommand should be cliConnection.CliCommand

drich10 added a commit that referenced this pull request Nov 12, 2014
edit cf CLI dev guide README
@drich10 drich10 merged commit 263b6fe into cloudfoundry:master Nov 12, 2014
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.

6 participants