Argument parsing #2

Merged
witten merged 4 commits from deroshkin/novel-stats-fork:deroshkin_branch into master 1 year 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
witten commented 1 year 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 year ago
witten referenced this issue from a commit 1 year ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as 7ee1c7c439.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b deroshkin-deroshkin_branch master
git pull deroshkin_branch

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff deroshkin-deroshkin_branch
git push origin master
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
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: witten/novel-stats#2
Loading…
There is no content yet.