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

Wrap CLI options in description with backticks #544

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

webknjaz
Copy link
Member

@webknjaz webknjaz commented Dec 1, 2022

This is necessary to make Sphinx display them as inline code and so it stops converting double dashes (--) to M-dashes ().

This is necessary to make Sphinx display them as inline code and
so it stops converting double dashes (`--`) to M-dashes (`—`).
@webknjaz
Copy link
Member Author

webknjaz commented Dec 1, 2022

I've also noticed weird section titles like https://pypa-build.readthedocs.io/en/stable/#python--m-build-options and posted tox-dev/sphinx-argparse-cli#68 upstream assuming it's their bug.

cc @gaborbernat

@layday
Copy link
Member

layday commented Dec 1, 2022

This doesn't seem to render correctly: https://pypa-build--544.org.readthedocs.build/en/544/

@webknjaz
Copy link
Member Author

webknjaz commented Dec 1, 2022

Ah, that's a bummer! I suppose that it's up to @gaborbernat to check this upstream, then?

Should we keep the PR open for now?

@layday
Copy link
Member

layday commented Dec 1, 2022

Sure.

@FFY00 FFY00 added documentation Improvements or additions to documentation enhancement New feature or request blocked labels Mar 17, 2023
@gaborbernat gaborbernat marked this pull request as draft February 1, 2024 15:38
@gaborbernat
Copy link
Contributor

Ah, that's a bummer! I suppose that it's up to @gaborbernat to check this upstream, then?

PR to fix that is welcome 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants