-
Notifications
You must be signed in to change notification settings - Fork 86
[README.md] Add detail description of oci-image-tool #48
Conversation
|
LGTM |
|
To avoid duplication, I'd rather link to the man pages and have the examples and such live in the man pages. For more background, see opencontainers/runtime-tools#47 and here. |
|
It's very useful to have this information in the README, we can have it in both locations. |
|
On Mon, Oct 24, 2016 at 08:55:12AM -0700, Chris Aniszczyk wrote:
Can you explain why? A line or two of context and then a link to the man page seems like it would be just as useful.
Without a strong reason to duplicate the information, that just gives you two places to keep in sync when we have future updates. |
|
A lot people simply read the README, I'm fine with a quick summary of the available commands and maybe a quick example, which then links to the man page for more info. |
|
On Mon, Oct 24, 2016 at 09:19:10AM -0700, Chris Aniszczyk wrote:
So we both agree on a line or two of summary and a man-page link (which is part of this current pull-request). In #5's c8c9708, I've submitted oci-image-tools(7) with one-line descriptions and references to per-tool man pages. This PR is doing something similar in the README, but also adding examples. This early in the ecosystem, I expect examples are going to be hard to keep current. But it's maintainer time to keep repeated examples in sync. If they feel that's a good use of their time, more power to 'em ;). |
|
I agree that customers want to find and read man pages conveniently, which are not linked to |
|
please rebase |
|
@vbatts rebased |
Signed-off-by: Lei Jitang <[email protected]>
|
It's redundant with #92 close |
Signed-off-by: Lei Jitang [email protected]