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

[CI:DOCS] Fix markdown tables on docs.podman.io #8335

Merged
merged 1 commit into from
Nov 16, 2020

Conversation

Luap99
Copy link
Member

@Luap99 Luap99 commented Nov 13, 2020

Sphinx with recommonmark cannot render markdown tables at all.
There is a python package called sphinx-markdown-tables which
adds the markdown table support to recommonmark.
https://pypi.org/project/sphinx-markdown-tables/

By utilising this package we don't have to change our doc format.

Signed-off-by: Paul Holzinger [email protected]

@rhatdan
Copy link
Member

rhatdan commented Nov 13, 2020

@edsantiago
Copy link
Member

@Luap99 do you have a way we can see results?

@edsantiago
Copy link
Member

PS that's not to say I don't believe it'll work, I'm just a careful reviewer who likes to be thorough. Nice work, thank you!

@Luap99
Copy link
Member Author

Luap99 commented Nov 13, 2020

@edsantiago

Install the dependencies

dnf install python-sphinx python-recommonmark
pip install sphinx-markdown-tables

Build the site

cd docs
make html

Watch the result

cd build/html/markdown/
xdg-open podman.1.html

@TomSweeneyRedHat
Copy link
Member

@Luap99 Thanks for putting this together. I saw your comment about sphinx in the issue, but didn't quickly see where to drop the changes in our site. I speak HTML (and being a weird InDUHvidual) prefer it to MD, so the other change was quick and took the middle man out for me. However if this works, I'm very happy to take it as clonking @edsantiago 's verification script alone was not good.

@TomSweeneyRedHat
Copy link
Member

FWIW, I'd to do the following to get the dependencies setup.

dnf install python-sphinx pip3
pip3 install python-recommonmark
pip3 install sphinx-markdown-tables

@TomSweeneyRedHat
Copy link
Member

but then the build failed in my sandbox....

@rhatdan
Copy link
Member

rhatdan commented Nov 13, 2020

Needs a rebase.

Sphinx with recommonmark cannot render markdown tables at all.
There is a python package called `sphinx-markdown-tables` which
adds the markdown table support to recommonmark.
https://pypi.org/project/sphinx-markdown-tables/

By utilising this package we don't have to change our doc format.

Signed-off-by: Paul Holzinger <[email protected]>
@rhatdan
Copy link
Member

rhatdan commented Nov 14, 2020

/approve

@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Luap99, rhatdan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rhatdan
Copy link
Member

rhatdan commented Nov 14, 2020

Fixes: #8317

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 14, 2020
@rhatdan
Copy link
Member

rhatdan commented Nov 16, 2020

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 16, 2020
@openshift-merge-robot openshift-merge-robot merged commit e593949 into containers:master Nov 16, 2020
@TomSweeneyRedHat
Copy link
Member

@Luap99 this did fix the table structure thanks!
However, the side bar on the home page is now a little askew. The "Join Community" button now has a return in it and it doesn't fit in the purple box now. I can't imagine this change would cause that, but I don't think we've had any other functional changes there recently. Thoughts?

@edsantiago
Copy link
Member

@TomSweeneyRedHat I assume by "home page" you mean https://podman.io/ ? (I see a "Join Community" button there in a purple box). If so, can you paste a screen shot and perhaps describe your browser? On my setup (Firefox) I can't get the "Join Community" text to spill out of the purple box no matter how narrow I make my browser window. (Snapshot below). But before spending more time on this, let's please all make sure we're talking about the same URL.
podman-join-community

@TomSweeneyRedHat
Copy link
Member

@edsantiago yes, that's the page, podman.io.
I just pulled the page up again to get a screen shot of it to add here, and it's displaying fine now in my newly updated Firefox. Prior il looked like there was a carriage return after the word "Join" and then "Community" was underneath is partially in the purple and mostly in the black.

So I think at this point it was just some kind of cache issue with Firefox at the time, so just ignore it, and sorry for the fire drill!

@Luap99 Luap99 deleted the fix-html-tables branch November 19, 2020 20:33
@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 24, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. 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.

6 participants