witten
/
novel-stats
1
0
Fork 1
Code Issues Pull Requests Releases Activity

Labels Milestones

Argument parsing #2

Merged
witten merged 4 commits from deroshkin/novel-stats-fork:deroshkin_branch into master 1 year ago
Conversation 1 Commits 4 Files Changed 2
deroshkin commented 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
Argument parsing' (#2)
continuous-integration/drone/pr Build is passing
Details
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.
Reviewers
Request review
No reviewers
Labels
Apply labels
Clear labels
No items
No Label
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
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
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'deroshkin/novel-stats-fork:deroshkin_branch'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes