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

Mostly cosmetic man 5 flashmq.conf improvements #123

Closed
wants to merge 2 commits into from

Conversation

bigsmoke
Copy link
Contributor

@bigsmoke bigsmoke commented Nov 5, 2024

See commit messages…

Rowan Rodrik van der Molen added 2 commits November 5, 2024 23:22
When there were alternative values for an option, these were oft
(1) separated by a slash, and (2) placed within a single `<replaceable>`
element. This (1) made them look very similar to path values, and (2)
made the values and their separator all end up with the same formatting.

Also, there were a couple of options for which the example/enum values
were not included in the `<term>` at all, but instead included in a
list, which, though not wrong in any way, I found to be inconsistent.
@halfgaar
Copy link
Owner

halfgaar commented Nov 7, 2024

Good stuff. Patches applied.

I had just edited the man page myself, so the generated files clashed. I edited your commits to only change the docbook file, applied the patches, rebase+edit the commits and rerun make per commit...

@halfgaar halfgaar closed this Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants