-
-
Notifications
You must be signed in to change notification settings - Fork 311
fix(cli): move sys.excepthook override to correct line, fix no argv test #1791
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
base: master
Are you sure you want to change the base?
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1791 +/- ##
=======================================
Coverage 97.95% 97.95%
=======================================
Files 60 60
Lines 2646 2646
=======================================
Hits 2592 2592
Misses 54 54 ☔ View full report in Codecov by Sentry. |
Lee-W
left a comment
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.
I thought this is handled in
Line 563 in aa82b98
| def commitizen_excepthook( |
|
Oh I see, will take a look |
0794acd to
4e29eb2
Compare
4e29eb2 to
70edfe0
Compare
| out, _ = capsys.readouterr() | ||
| assert out.startswith("usage") |
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.
These lines are unreachable...
53b0f25 to
8718994
Compare
… parameter, fix no argv test
8718994 to
5d6f91a
Compare
Closes #1789