[New] max-dependencies: add option ignoreTypeImports

[rfermann]

Adds support for ignoring type imports.

Fixes #1843. Fixes #1136.

[coveralls]

Coverage decreased (-0.2%) to 97.695% when pulling b11f5ab on rfermann:1843 into 843055c on benmosher:master.

[coveralls]

Coverage increased (+0.005%) to 97.862% when pulling 8195aa3 on rfermann:1843 into 843055c on benmosher:master.

[coveralls]

Coverage increased (+0.005%) to 97.862% when pulling 8195aa3 on rfermann:1843 into 843055c on benmosher:master.

[coveralls]

Coverage increased (+0.005%) to 97.862% when pulling 8195aa3 on rfermann:1843 into 843055c on benmosher:master.

[coveralls]

Coverage increased (+0.005%) to 97.862% when pulling 8195aa3 on rfermann:1843 into 843055c on benmosher:master.

[cherryblossom000]

This will also fix #1136, won't it?

[ljharb]

Thanks! You added two test cases with babel-eslint, which covers flow - could we also add some that use TS?

[rfermann]

@ljharb I added some tests using the TS parser. But as syntax for type import looks identical for flow and TS, I am not sure if that is what you were asking for.

[ljharb]

Thanks, LGTM pending the one comment

[cherryblossom000]

Hi, I'm running into this issue a lot and keep having to put // eslint-disable-next-line import/max-dependencies for cases where the type imports make the number of imports exceed the configured max amount. It'll be nice if this PR could get a rebase and be merged.

[himynameisdave]

Would be nice to document this option over here.