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
make the man page section selectable #1227
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I strongly recommend adding the 8 as a "default" for people who have an understanding of what the section number represents
@lynncyrin I respect your opinion about the default section number. Since Go does not support the optional argument, I would like to work on one of the following implementations.
|
This one sounds best! |
a6a5f77
to
25e94ac
Compare
@lynncyrin Thanks! I've submitted a new patch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you ✨ can you also increase the code coverage? This page will help
25e94ac
to
9d658a4
Compare
Signed-off-by: Nobuhiro MIKI <nob@bobuhiro11.net>
9d658a4
to
ed2ee4b
Compare
@lynncyrin Coverage has improved. Can you please check again? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's nice, LGTM 👍
What type of PR is this?
What this PR does / why we need it:
The section number of the man page is fixed to 8.
This number can take other values depending on the usage.
https://www.man7.org/linux/man-pages/man7/man-pages.7.html
This fix adds a new argument to the
ToMarkdown
andToMan
methods so that they can be set to arbitrary values.If you prefer other implementation, please let me know.
Testing
Existing test code has been reused.
Release Notes