rffontenelle/powrap
1
0
Fork 0
forked from AFPy/powrap
Code Pull requests Activity

Improve parse_args #1

Closed
rffontenelle wants to merge 1 commit from rffontenelle-improve-parse_args into main
pull from: rffontenelle-improve-parse_args
merge into: rffontenelle:main
Conversation 0 Commits 1 Files changed 1 +4 -3
rffontenelle commented 2023-06-03 17:43:13 +02:00
Owner
Copy link

This solves two minor issues of the powrap --help output:

  • First one is that argparse shows short and then long form of --help option, while powrap's arguments show long and then short, causing it to look like:
options:
  -h, --help      show this help message and exit
  --modified, -m  Use git to find modified files ...
  -C GIT_ROOT     To use with --modified to tell ...
  --quiet, -q     Do not show progress bar ...
  ...
  • The other thing is that the exit code is set in multi-line string, but not actually printed as multi-line string. While it looks compacted and OK in the printed help message, I assume that's not the intended so I set RawTextHelpFormatter to make sure the formatting is honored.
exit code: 0:nothing to do 1:would rewrap 127:error running msgcat

Signed-off-by: rffontenelle rffontenelle@gmail.com

This solves two minor issues of the `powrap --help` output: - First one is that argparse shows short and then long form of --help option, while powrap's arguments show long and then short, causing it to look like: ``` options: -h, --help show this help message and exit --modified, -m Use git to find modified files ... -C GIT_ROOT To use with --modified to tell ... --quiet, -q Do not show progress bar ... ... ``` - The other thing is that the exit code is set in multi-line string, but not actually printed as multi-line string. While it looks compacted and OK in the printed help message, I assume that's not the intended so I set RawTextHelpFormatter to make sure the formatting is honored. ``` exit code: 0:nothing to do 1:would rewrap 127:error running msgcat ``` Signed-off-by: rffontenelle <rffontenelle@gmail.com>
Signed-off-by: rffontenelle <rffontenelle@gmail.com>
rffontenelle closed this pull request 2023-06-03 17:43:35 +02:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
rffontenelle/powrap!1
No description provided.