Check fails for submodules

[m-aciek]

Current behaviour:

$ cat positive.py 
from collections.abc import Container

Container()
$ flake8 positive.py 
positive.py:1:1: I801 use 'import collections.abc' instead of 'from collections.abc import Container'
$
$ cat false-positive.py 
from collections import abc

abc.Container()
$ flake8 false-positive.py 
false-positive.py:1:1: I801 use 'import collections' instead of 'from collections import abc'
$
$ cat negative.py 
import collections

collections.abc.Container()
$ flake8 negative.py 
$ 

Checks should pass for second file, shouldn't they?

[sfstpala]

No, it actually wants you to type import collections.abc instead of from collections import abc.

I see where you're coming from, and it makes sense. Maybe it should be an option at least. The convention that birthed this module was basically to never use from at all, wether you're importing a module or another object.

However, the error message is definitely wrong. It should say use 'import collections.abc' instead of 'from collections import abc'

[mnowotka]

Ability to configure the project and relax some constraints would be super useful!