We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
There is the following lines of code in import_params.py.
import_params.py
Generally "bare excepts" like this are discouraged:
There is even PEP760 which proposes disallowing them altogether.
https://peps.python.org/pep-0760/
I suggest we find out what all the branches are and treat them explicitly instead of a catch-all try except.
try except
The text was updated successfully, but these errors were encountered:
Not only is it a bare except, but it is also letting w/e exceptions occur to pass silently:
https://stackoverflow.com/questions/21553327/why-is-except-pass-a-bad-programming-practice
https://peps.python.org/pep-0020/
Sorry, something went wrong.
No branches or pull requests
There is the following lines of code in
import_params.py
.Generally "bare excepts" like this are discouraged:
There is even PEP760 which proposes disallowing them altogether.
https://peps.python.org/pep-0760/
I suggest we find out what all the branches are and treat them explicitly instead of a catch-all
try except
.The text was updated successfully, but these errors were encountered: