FBT001 exclude operators like __and__, __or__ and __xor__, etc.

[randolf-scholz]

FBT001 should probably by default exclude methods used for operator overloading, in particular boolean logic specific operations:

• __and__, __rand__, __iand__
• __or__, __ror__, __ior__
• __xor__, __rxor__, __ixor__

And potentially also arithmetic operations like __add__, __sub__, __mul__, etc. (These may add overloads / type hints for boolean values)

[dylwil3]

This is an interesting idea! I guess one question I have is: if you are defining one of the Boolean magic methods for MyClass, why would the other parameter be a bool (as opposed to MyClass? Similarly for arithmetic operations.

I'd be interested to see examples of this in code out in the wild.

[MichaReiser]

An other option would be to ignore methods marked with @overload

[randolf-scholz]

@dylwil3 A typical example are custom scalar/vector/tensor classes/protocols.

For instance, a BooleanArray might define __or__(self, other: bool | Self) -> Self.

[dylwil3]

Alright that makes sense to me- thanks for the example! I'm in favor of adopting this change.