Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

portusctl: added man pages #1021

Merged
merged 1 commit into from
Aug 10, 2016
Merged

portusctl: added man pages #1021

merged 1 commit into from
Aug 10, 2016

Conversation

mssola
Copy link
Collaborator

@mssola mssola commented Aug 5, 2016

See #868

Signed-off-by: Miquel Sabaté Solà [email protected]

@mssola mssola changed the title portusctl: added man pages [WIP] portusctl: added man pages Aug 5, 2016
@mssola
Copy link
Collaborator Author

mssola commented Aug 5, 2016

@mssola mssola mentioned this pull request Aug 9, 2016
2 tasks
@mssola
Copy link
Collaborator Author

mssola commented Aug 9, 2016

@jordimassaguerpla @flavio @monstermunchkin @tboerger PTAL (tip: take a look at the md files not the .1 files, since they are generated from their correspondent .md files 😉 )

I'll squash my commits once the review is over.

@mssola mssola changed the title [WIP] portusctl: added man pages portusctl: added man pages Aug 9, 2016
```

Note that this should be done by developers. Packagers should only have to care
about the files inside of `man/man1` and disregard completely the markdown files.
Copy link
Contributor

Choose a reason for hiding this comment

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

"… and disregard the markdown files completely."

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

@jordimassaguerpla
Copy link
Member

I've looked into the spec file and then I did a quick look at the rest.
LGTM


# DESCRIPTION
**portusctl** is a tool that helps administrators to perform operation on the
installed Portus instance. For this, administrators will be able to execute
Copy link
Contributor

Choose a reason for hiding this comment

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

"… helps administrators perform operations on …"

@monstermunchkin
Copy link
Contributor

Other than the comments, LGTM.

@flavio
Copy link
Member

flavio commented Aug 10, 2016

I think @monstermunchkin's comments covered everything. LGTM

Fixes SUSE#868

Signed-off-by: Miquel Sabaté Solà <[email protected]>
@mssola mssola merged commit 3499bae into SUSE:master Aug 10, 2016
@mssola mssola deleted the man branch August 10, 2016 15:26
==============================================================

# NAME
portusctl logs \- Fetch all the logs produced by Portus
Copy link
Contributor

Choose a reason for hiding this comment

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

It's portusctl exec and not logs markdown file.

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

Successfully merging this pull request may close these issues.

5 participants