-
Notifications
You must be signed in to change notification settings - Fork 46
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
deprecate -q parameter in CLI #215
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like this PR is built on the branches pv/aoi-example
and pv/default-max-items
-- I'm guessing that was accidental?
no, i didn't want to have to worry about merging -- I meant to make this a draft pr |
Co-authored-by: Pete Gadomski <pete.gadomski@gmail.com>
clearly i blacked-out while writing this help message Co-authored-by: Pete Gadomski <pete.gadomski@gmail.com>
Codecov Report
@@ Coverage Diff @@
## main #215 +/- ##
==========================================
- Coverage 83.75% 83.18% -0.57%
==========================================
Files 9 9
Lines 677 684 +7
==========================================
+ Hits 567 569 +2
- Misses 110 115 +5
Continue to review full report at Codecov.
|
Related Issue(s): #178
Description:
Deprecate -q parameter, as it's the only single-letter parameter other than -u (url) and can easily be confused with quiet
PR Checklist: