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

add menu.md, make index.md command line reference, update typos/minor… #24059

Merged
merged 1 commit into from
Jul 1, 2016

Conversation

sfsmithcha
Copy link
Contributor

implement menu.md for Engine CLI docs. Put CLI links into a table with short description. correct node_inspect to refer to node instead of volume. Fix minor typos.

Signed-off-by: Charles Smith charles.smith@docker.com

@sfsmithcha
Copy link
Contributor Author

ping @thaJeztah

* [network_disconnect](network_disconnect.md)
* [network_inspect](network_inspect.md)
* [network_ls](network_ls.md)
* [network_rm](network_rm.md)
Copy link
Member

Choose a reason for hiding this comment

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

oh, the link-captions shouldn't have the underscores (also below for volume, node)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah that was there already and I was thinking it was off when I was working on it. Will fix.

@sfsmithcha sfsmithcha force-pushed the fix_cli_ref_menu branch 2 times, most recently from f57e486 to b04ab16 Compare June 30, 2016 21:27
@sfsmithcha
Copy link
Contributor Author

sfsmithcha commented Jun 30, 2016

@thaJeztah PTAL. Good thing you asked me to clean up the formatting. Discovered I had a copy/paste error in volume command section. I also removed the single links from the menu page and referenced the index.md (so there wouldn't be 2 places to maintain).

… errors in cli docs

Signed-off-by: Charles Smith <charles.smith@docker.com>
@thaJeztah
Copy link
Member

LGTM, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants