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

Report error when bare ExceptionClass() used instead of raise ExceptionClass() #7189

Closed
socketpair opened this issue Jul 14, 2022 · 2 comments
Labels
Duplicate 🐫 Duplicate of an already existing issue

Comments

@socketpair
Copy link

socketpair commented Jul 14, 2022

Current problem

code like

def some_action():
    RuntimeError()

is definitely an error

PyLint should detect and issue a message.

@socketpair socketpair added the Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling label Jul 14, 2022
@Pierre-Sassoulas Pierre-Sassoulas added Proposal 📨 Needs decision 🔒 Needs a decision before implemention or rejection and removed Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling labels Jul 14, 2022
@Pierre-Sassoulas
Copy link
Member

Thank you for opening the issue but please use the template. You have to actually detail what is the problem and what is the solution (specify what the message name should be and what is the actual problem) so there is less work for readers / implementers.

@DanielNoord
Copy link
Collaborator

Duplicate of #3110

@DanielNoord DanielNoord added Duplicate 🐫 Duplicate of an already existing issue and removed Proposal 📨 Needs decision 🔒 Needs a decision before implemention or rejection labels Jul 14, 2022
@DanielNoord DanielNoord closed this as not planned Won't fix, can't repro, duplicate, stale Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Duplicate 🐫 Duplicate of an already existing issue
Projects
None yet
Development

No branches or pull requests

3 participants