B027 and typing.overload

Consider the following:

class Foo(ABC):
    @t.overload
    def bar(self, arg: t.Literal[False]) -> t.Literal[False]:
        ...

    @t.overload
    def bar(self, arg: t.Literal[True]) -> t.Literal[True]:
        ...

    def bar(self, arg: bool) -> bool:
        return arg

    @abstractmethod
    def abstract_method(self) -> None:
        ...

The overloads of bar are considered as “real” methods and B027 warning is emitted:

 B027 bar is an empty method in an abstract base class, but has no abstract decorator. Consider adding @abstractmethod.
 B027 bar is an empty method in an abstract base class, but has no abstract decorator. Consider adding @abstractmethod.

Is it worth not emitting the warning when typing.overload is used or is it a case that should be supressed manually with # noqa?

[jakkdl]

yep!

[jakkdl]

Typecheckers evaluate imports, but AST checkers like flake8 usually don’t try and just looks at identifiers hoping/assuming they’re well-named. (afaik)

Anyway, fixed with #309 - forgot to mention this issue in it.