Skip to content
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

Tighten up bare excepts around the codebase (would also fix E722 warnings) #48

Open
Horgix opened this issue May 3, 2021 · 0 comments
Labels
refactoring Stuff to refactor (without features impact) tooling/lint #52B06B

Comments

@Horgix
Copy link
Collaborator

Horgix commented May 3, 2021

There are 9 bare except: in the codebase.

I won't explain why it's discouraged, because Flake8 F722 does it well already.

This could be solved by either really thinking about what the Exceptions trying to be caught are, or by using except Exception: instead to improve things slightly.

@Horgix Horgix added tooling/lint #52B06B refactoring Stuff to refactor (without features impact) labels May 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Stuff to refactor (without features impact) tooling/lint #52B06B
Projects
None yet
Development

No branches or pull requests

1 participant