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
chore: update changelog, minor touchups #469
Conversation
print('{red}ERROR{reset} {}'.format(msg, **_STYLES)) | ||
exit(code) | ||
print('{red}ERROR{reset} {}'.format(msg, **_STYLES), file=sys.stderr) | ||
raise SystemExit(code) |
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.
exit()
is not part of the language, it's injected by site.py
. Better to use sys.exit()
, or even better, just raise SystemExit
directly.
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.
Yes, that is a bug.
] | ||
|
||
|
||
def __dir__() -> List[str]: |
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.
Makes tab completion prettier on Python 3.7+.
I assume the tests are failing because errors are printed to |
Co-authored-by: layday <layday@protonmail.com> Apply suggestions from code review tests: add test for coverage
2eb4794
to
76c30c9
Compare
Added a test for |
A few fixes and cleanups.
IsolatedEnv
#361