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

podman-run: clean up some formatting issues #911

Closed
wants to merge 1 commit into from

Conversation

miabbott
Copy link
Contributor

@miabbott miabbott commented Jun 6, 2018

I was browsing this page from the repo and some of the formatting was ugly.

This is a shot at cleaning some of it up.

Rendered version can be seen here: /~https://github.com/miabbott/libpod/blob/fmt-podman-run/docs/podman-run.1.md

@rhatdan
Copy link
Member

rhatdan commented Jun 6, 2018

@miabbott Sign your PR, please.

@rhatdan
Copy link
Member

rhatdan commented Jun 6, 2018

@miabbott Did you verify these still look good as man pages as well as when viewed in markdown?

@miabbott
Copy link
Contributor Author

miabbott commented Jun 6, 2018

@rhatdan I did not....so this PR is probably going to need more work

Signed-off-by: Micah Abbott <miabbott@redhat.com>
@miabbott
Copy link
Contributor Author

miabbott commented Jun 6, 2018

@rhatdan Looks like I got lucky with my changes...man page looks reasonable, nothing broken AFAICT.

I force pushed a new commit that I've signed.

@rhatdan
Copy link
Member

rhatdan commented Jun 6, 2018

LGTM @TomSweeneyRedHat PTAL

Copy link
Member

@TomSweeneyRedHat TomSweeneyRedHat left a comment

Choose a reason for hiding this comment

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

LGTM, @miabbott thx a bunch for all the touch ups!

@rhatdan
Copy link
Member

rhatdan commented Jun 6, 2018

@rh-atomic-bot r+

@rh-atomic-bot
Copy link
Collaborator

📌 Commit d60e541 has been approved by rhatdan

@rh-atomic-bot
Copy link
Collaborator

⌛ Testing commit d60e541 with merge 16c09f8...

# podman run -it --blkio-weight-device "/dev/sda:200" ubuntu

```
$ podman run -d busybox top
Copy link
Contributor

Choose a reason for hiding this comment

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

Was this example intentionally dropped? It's originally from 8cf07b2 (#2) in case that helps.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It didn't look like it fit in with the --blkio-weight-device example.

I could be misunderstanding how that example works, though.

Copy link
Member

@TomSweeneyRedHat TomSweeneyRedHat Jun 6, 2018

Choose a reason for hiding this comment

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

@wking I was OK removing it as I didn't see how it applied to the verbiage above it at line 793 does. @mheon @baude any thoughts?

@rh-atomic-bot
Copy link
Collaborator

☀️ Test successful - status-papr
Approved by: rhatdan
Pushing 16c09f8 to master...

@github-actions github-actions bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 27, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants