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

integrated command line with argparse library #54

Merged
merged 5 commits into from
May 27, 2023

Conversation

H4ppy-04
Copy link
Owner

  • Adds boilerplate code (--help) with argparse
  • --index, --author and --cols command
  • Takes word as required argument
  • Backwards compatible!

* Add boilerplate code for parser

* Index command (no functionality yet)

* Filter author command (no functionality yet)

Issue: #53

Authored-by: Joshua Rose <[email protected]>
@H4ppy-04 H4ppy-04 added the enhancement New feature or request label May 27, 2023
@H4ppy-04 H4ppy-04 linked an issue May 27, 2023 that may be closed by this pull request
* implement --cols paramater

* It half works, need to work on the formatter function

* Don't touch formatter.

Authored-by: Joshua Rose <[email protected]>
@codecov
Copy link

codecov bot commented May 27, 2023

Codecov Report

Merging #54 (380363d) into main (5e75353) will decrease coverage by 36.67%.
The diff coverage is 16.04%.

@@             Coverage Diff             @@
##             main      #54       +/-   ##
===========================================
- Coverage   82.66%   45.99%   -36.67%     
===========================================
  Files           1        1               
  Lines         248      337       +89     
===========================================
- Hits          205      155       -50     
- Misses         43      182      +139     
Impacted Files Coverage Δ
src/urban.py 45.99% <16.04%> (-36.67%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

* Adds interation for `--cols`

* Enforces `args.words` as argument instead of `sys.argv`

Issue: #53
Pull Request: #54
Tag: 1.1.0

Authored-by: Joshua Rose <[email protected]>
* Tests formatting empty definitions

* Tests single word definitions

* Blacken code

Authored-by: Joshua Rose <[email protected]>
@H4ppy-04 H4ppy-04 merged commit 25939c0 into main May 27, 2023
H4ppy-04 added a commit that referenced this pull request May 27, 2023
* Adds interation for `--cols`

* Enforces `args.words` as argument instead of `sys.argv`

Issue: #53
Pull Request: #54
Tag: 1.1.0

Authored-by: Joshua Rose <[email protected]>
* Omits to-be-added funcionts (via block comments)

Authored-by: Joshua Rose <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integrate command line with the argparse library
1 participant