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.

Sign up for GitHub

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

Jump to bottom

Request: checking for empty Exception messages #13

Closed
jamesbraza opened this issue Apr 28, 2023 · 0 comments · Fixed by #23
Closed

Request: checking for empty Exception messages #13

jamesbraza opened this issue Apr 28, 2023 · 0 comments · Fixed by #23

Comments

@jamesbraza
Copy link

jamesbraza commented Apr 28, 2023 •
edited
Loading

If one throws an Exception like so:

raise Exception
# of
raise Exception()

Then the Exception will show up in log files as a blank line (if using %(message) in logging formatter), which will eventually result in debugging headaches.

I think a checker for blank Exception messages would be useful (at least for me). Thank you!
@jamesbraza jamesbraza changed the title Request: checking for empty Exceptions Request: checking for empty Exception messages Apr 28, 2023
@henryiii henryiii mentioned this issue Oct 3, 2023
Merged
henryiii added a commit that referenced this issue Oct 3, 2023
@henryiii
feat: add checks for built-in exceptions (#23)
efb36a6 
Closes #13. This can only be done for built-in exceptions, since there's
no way to tell if a variable is a class or an object.

Signed-off-by: Henry Schreiner <[email protected]>
@opk12 opk12 mentioned this issue Sep 25, 2024
Open
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add checks for built-in exceptions henryiii/flake8-errmsg
1 participant
@jamesbraza