unawaited_futures: False positives -> '.then'

[matanlurey]

This should not trigger a lint warning:

new File('someFile.txt').readAsString().then(print);

But it does. On top of it, I can't ignore the lint warning at all:

// ignore: unawaited_futures
new File('someFile.txt').readAsString().then(print);

If this can't be fixed I'm request the lint be disabled for the next SDK release.

[matanlurey]

EDIT: Looks like restarting analysis does "fix" it, temporarily. New warnings are not ignorable until you restart again, though, and it sort of still presents itself as you code/add more.

[matanlurey]

Ping.

This is really annoying. I'd prefer this fixed before new lints are added if that's reasonable.

[alexeieleusis]

The new lints you are seeing are part of a project trying to add the readability/effective dart related lints. Once they are added (the machine verifiable ones of course) some other improvements will be triaged, but so far we are planning on adding support for those rules in the analysis server to have the corresponding quick fixes available in intellij and other editors. Since this project mostly lives of community contributions, it is a good idea to reach directly to the author, I believe it to be @ochafik.

[matanlurey]

Where is the quick fix support? That's part of the linter?

[bwilkerson]

Quick fixes lives in the analysis_server package.

[natebosch]

@bwilkerson - is there a reasonable chance of getting a quickfix that wraps the future with a call to unawaited from package:pedantic?

[bwilkerson]

I have no problem having such a fix added; it's merely a question of priorities and resources.

[matanlurey]

I haven't seen an issue with this.