manpage: Split out options by argument group.#35
Conversation
|
Seems this is failing tests - will fix. |
64997e4 to
133eae2
Compare
|
OK so fixed the failing tests, mostly by changing the expected output. Now Also, options not in an argument group are put below subcommands. Is that bad? Thanks. |
|
Thank you for the PR! See the example Also, the word |
133eae2 to
aaa8105
Compare
|
Fixed re global options movement.
? |
|
Yes, either keep "Sub-commands", or "ACTIONS". If I understand your change correctly, |
46ec60c to
405a702
Compare
|
I kept the title (here I'll let you decide if you want to make changes to that they are both .SH or .SS and commit that. I've created a few more commits in this branch you might be interested in too https://github.com/wwood/argparse-manpage/tree/bird_opinions - most are self-contained in single commits if you wanted to take them. Some border more closely align with my opinions though so may not be interesting. |
|
Also note I changed the instantiation of the copr so that the default (without title) was being tested, intentionally. |
|
Thank you, I briefly checked the |
|
I'm happy enough with my contributions so far, particularly since I've a special use-case for these man pages and don't require the full code base to be functional meaning I'll have to maintain a fork anyway. Still, I'm happy to answer questions - just shoot me an email or tag me here or something. Thanks. |
Accepted. Thank you anyway! |
|
@wwood, there was some problem I don't yet understand, but your patch was breaking |
|
I hope I finally understood the hierarchy, so I rather made a bit more changes to make this correct. |
Hi again,
Here I introduce some opinion, that the manpage should separate the options by argument group rather than putting them all under the OPTIONS heading. For programs with lots of options it helps to organise them e.g. like https://wwood.github.io/CoverM/coverm-genome.html
Thoughts?