Fix flow import type

[jwangnz]

import type Foo from 'foo';

with eslint 0.21.0, the code above will cause error:

error  Foo is defined but never used  no-unused-vars

[sebmck]

Isn't this error correct? You need to use the Foo type somewhere?

[jwangnz]

@sebmck

import type Foo from 'foo';

export function bar(foo: Foo) {
  console.log(foo);
}

In the code above, type Foo is used, but eslint still complain Foo is defined but never used

[sebmck]

Not sure, I'll have to think about it. I'd rather also lint type annotations if possible.

[jwangnz]

@sebmck Do you have any ideas about this ?

[sebmck]

Ideally flow types should be retained (ie. remove this line) and ESLint internals patched to crawl the type annotations and report accurately on them.

[jwangnz]

@sebmck It seems like a big work to do, I'm noob on this kind of AST things.

Do you have plan to implement this ?

Or could you consider merge this for now, since there is already a line remove all the type annotations?

[jwangnz]

close this as @sebmck point out this may break peoples linting as some people have it setup with their testing frameworks etc.