bellybutton icon indicating copy to clipboard operation
bellybutton copied to clipboard

Feature suggestion: Ignore only given set of rules

Open tomascapek opened this issue 7 years ago • 2 comments

In my usage of this awesome tool I found out, that # bb: disable can sometimes hide other errors, which would be otherwise addressed. Let's consider following example:

deprecated_fn(42 + inner_fn())

So one day, first function became deprecated and thus rule DeprecatedFnCall was created. For some reason a programmer decided, that he still needs to keep that old deprecated function and so he disabled the warning.

deprecated_fn(42 + inner_fn()) # bb: disable

But the other day, inner_fn became deprecated too and I think you can see the problem now. If he'd been allowed to do something similar to what pylint does:

 deprecated_fn(42 + inner_fn()) # bb: disable = DeprecatedFnCall

With the creation of the new rule, he would be able to detect this fatal problem, because it would immediately show up as an error.

Of course, this is you project and this is only a suggestion :)

I am also willing to create this feature myself, if you like this idea and don't {want | have time} to deal with it.

tomascapek avatar Aug 14 '18 13:08 tomascapek

Hi @tomascapek , I will plan on adding this feature for v.0.2.4.

hchasestevens avatar Aug 17 '18 02:08 hchasestevens

Hello, @hchasestevens. Any idea when you are going to release it?

tomascapek avatar Aug 17 '18 15:08 tomascapek