Argument parsing #2

Merged
witten merged 4 commits from deroshkin/novel-stats-fork:deroshkin_branch into master 1 month ago

Hey, I got around to improving the argument parsing (largely to force myself to learn it).

Also 2 other minor changes:

  1. Switched act summary print statement to the newer string formatting.
  2. Fixed my mistake in the readme, flag should have been -pp, it was --pp in one spot, also since the order of arguments no longer matters I switched the order to the more standard filename at the end.

tox comes back good on python 3.9 on debian

Hey, I got around to improving the argument parsing (largely to force myself to learn it). Also 2 other minor changes: 1. Switched act summary print statement to the newer string formatting. 2. Fixed my mistake in the readme, flag should have been `-pp`, it was `--pp` in one spot, also since the order of arguments no longer matters I switched the order to the more standard filename at the end. `tox` comes back good on python 3.9 on debian
deroshkin added 4 commits 1 month ago
Owner

Looks awesome! Thank you for taking the time to do this.

Looks awesome! Thank you for taking the time to do this.
witten merged commit 7ee1c7c439 into master 1 month ago
witten referenced this issue from a commit 1 month ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as 7ee1c7c439.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.