Check for raise Exception or BaseException

[avylove]

Current problem

Linting a codebase with a lot of raise Exception() statements. This is a bad practice as it makes it impossible to write narrow exception clauses.

Desired solution

Forgive me if I missed it, but as far as I can tell, a check doesn't currently exist. The check should fail when an instance of Exception or BaseException is raised directly.

[Pierre-Sassoulas]

That would be broad-except, can you try to enable it ?

[avylove]

Isn't broad-except on the catch side, not the raise side?

[Pierre-Sassoulas]

Ha, right !

[clavedeluna]

What should the linter name be here, broad-raise?

[Pierre-Sassoulas]

I'm thinking that we could rename broad-except to broad-exception-caught and name this one broad-exception-raised. We need to use 'old-names' for that.

[clavedeluna]

'old-names' for that.

That's a way to handle backward compat, right? Or since this is a breaking change would this only be released in pylint 3.x? Trying to understand where to put this PR to

[Pierre-Sassoulas]

Yes, old names handle the backward compatibility so that it's not a breaking change. See C0111 /missing-docstring for example.