Skip to content

Change top level heading in man pages to upper case for consistency in Open MPI man pages. #11077

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

Merged
merged 1 commit into from
Nov 15, 2022
Merged

Conversation

drwootton
Copy link
Contributor

Most man page source files in the docs/man-openmpi/man3 sub directory have top levels headings such as SYNTAX, ERRORS, NOTES, etc. Some have these headings, but in mixed case.

This pull request changes top level headings to upper case for consistency within the set of man pages, and because other sets of man pages like libc man pages and some other GNU man pages use upper case for top level section headings.

Signed-off-by: David Wootton [email protected]

in Open MPI man pages.

Signed-off-by: David Wootton <[email protected]>
@drwootton drwootton requested a review from jjhursey November 15, 2022 20:15
@drwootton drwootton changed the title Change top level heading in man pages to upper case for consistencyin Open MPI man pages. Change top level heading in man pages to upper case for consistency in Open MPI man pages. Nov 15, 2022
Copy link
Member

@jsquyres jsquyres left a comment

Choose a reason for hiding this comment

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

Thank you!

Copy link
Member

@jjhursey jjhursey left a comment

Choose a reason for hiding this comment

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

Awesome. Thank you!

@jjhursey jjhursey merged commit 0119b5a into open-mpi:main Nov 15, 2022
@jjhursey
Copy link
Member

Please PR to v5 when you have a chance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants