jdufresne / flake8-raise Goto Github PK

@jdufresne Thanks for your plugin, I would like to offer one more check about exceptions

R102 Unnecessary parentheses in raise exception

if something:
    raise PermissionDenied()

Will result in the error:
R102 Unnecessary parentheses in raise exception

To fix, change to:

if something:
    raise PermissionDenied

If there is no difference then why write more?
However, there may be people who prefer the opposite so that the brackets are always there and you can do the reverse optional check.

I would like to propose another check.

R103 Missing parentheses on raised exception

It would be off by default, but could be activated for projects that want to follow convention with having always parentheses (see google style docs).

if something:
    raise PermissionDenied

Will result in the error:
R103 Missing parentheses on raised exception

To fix, change to:

if something:
    raise PermissionDenied()

By the way, thanks for the plugin :)