-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Remove bare except usages #3137
Comments
So one needs to find all the lines where bare excepts are used using the above mentioned script and delete it, Am I right? |
I would like to work on this issue! |
Here is an example of bare exception:
Eliminating bare except is not deleting those lines, like replacing the above line with only
On what is wrong with bare except, see this SO answer and PEP-8. |
Mind if i have a crack at it? |
Hey @arunppsg If this issue is still open I would like to work on it! |
@arunppsg I would love to work on this issue if it is still available. |
Some parts of the code uses bare except clauses which should be replaced with valid exception clauses. To find the usages of bare except, edit
setup.cfg
and remove the flake8 ignore code line forE722
and run./scripts/flake8_for_ci.sh
. This will raise flake8 errors at the lines of code where bare excepts are used.The text was updated successfully, but these errors were encountered: