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

Use uniform formatting for SEE ALSO sections #2732

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

ffesti
Copy link
Contributor

@ffesti ffesti commented Oct 24, 2023

While there is no generally agreed standard for this we should at least use only one. Comma separated lists are commonly used. Let's just pick italic for man pages.

Thanks to Frank Dana https://github.com/ferdnyc for pointing this out.

Resolves: #2731

@ffesti
Copy link
Contributor Author

ffesti commented Oct 24, 2023

Improved wording of the commit message

@pmatilai
Copy link
Member

Hmm, on a random sampling (strtok, ls, systemd-inhibit, ssh, bash, auditd, getopt, grep, xhost, vim), bold was by far the most common style. That's what "man man" also uses, which one could think of as the "canonical source of truth" I guess. Italics (or underline as it renders for me) is typically used for paths.

While there is no generally agreed standard for formatting SEE ALSO
section of man pages we should at least use only one style. Comma separated
lists are commonly used when referring to other man pages. Both *italic* and
**bold** are used but the later seems to be more popular - so let's
stick to that.

Thanks to Frank Dana <https://github.com/ferdnyc> for pointing this out.

Resolves: rpm-software-management#2731
@ffesti
Copy link
Contributor Author

ffesti commented Oct 25, 2023

I don't have a strong opinion. Guess it depends on what pages you sample. I just switched to bold.

@pmatilai
Copy link
Member

I don't have a strong opinion either, this was just a statistical observation 😄

@pmatilai pmatilai merged commit c97fd93 into rpm-software-management:master Oct 26, 2023
1 check passed
@ferdnyc
Copy link
Contributor

ferdnyc commented Oct 31, 2023

@pmatilai

I don't have a strong opinion either, this was just a statistical observation 😄

And therefore technically correct. ("The best kind of correct!")

@ffesti ffesti deleted the seealso branch March 19, 2024 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Inconsistency in manpage formatting
4 participants