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

Compatibility with argparse-manpage #305

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ijc
Copy link

@ijc ijc commented Jan 8, 2023

https://pypi.org/project/argparse-manpage/

  • Refactor construction of argparse.ArgumentParser into a get_argparser() function in each binary.
  • Set prog property (the default uses sys.arg[0] which breaks with argparse-manpage, arguably this is an argparse-manpage bug).
  • Set the man_short_description property based on the first line of the description. This is used as a summary in the NAME section of the man page.

@ijc ijc force-pushed the support-argparse-manpage branch from e86d073 to 17fd2c9 Compare January 8, 2023 10:56
ijc added 2 commits January 8, 2023 10:58
* Refactor construction of `argparse.ArgumentParser` into a `get_argparser()`
  function in each binary.
* Set `prog` property (the default uses `sys.arg[0]` which breaks with
  `argparse-manpage`, arguably this is an `argparse-manpage` bug).
* Set the `man_short_description` property based on the first line of the
  description. This is used as a summary in the `NAME` section of the man page.
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.

1 participant