Lint for import "sort order". about linter

[harryterkelsen]

Yes please! The linter should complain if:

• imports are not correctly grouped into dart:, package: and relative imports
• groups are not separated by a blank line
• imports are not sorted alphabetically within groups

[trentgrover-wf]

any further plans on this?

[seaneagan]

Hopefully we can get this in the style guide too: dart-archive/www.dartlang.org#635

[pq]

Yes please. And thanks for the link @seaneagan. I was just looking for it myself! :)

[pq]

any further plans on this?

Sorry. Haven't gotten to it but it's definitely a good idea and doable. Added to a new tracking milestone NEXT.👍

[alexeieleusis]

I think that with this week's contributions we can close this.

[rdehouss]

Hi,

Sorry to get this one out after so long, but I don't think dartfmt does sort the import, right?
Should this ticket be re-opened?

Regards,

Raphaël

[pq]

@rdehouss: I think what you're after was implemented in directives_ordering. Let us know if that works for you!

[rdehouss]

Hi @pq, no, I was coming from dart-lang/sdk#20244 looking for a solution like https://github.com/timothycrosley/isort to sort automatically the import so that I don't have to fix each lint issues manually. But I understand that I'm now in the linter project and left dartfmt, sorry

[pq]

Ah. Got it. What we need here minimally is a quickfix for directives_ordering (as noted in #1374) and ideally support in something like dartfix so that there's more of a CLI for bulk application of the fix.

/fyi @bwilkerson

[bwilkerson]

The implementation of the quick fix should be largely the same as the implementation of the edit.organizeDirectives request. It should be fairly easy to refactor the code to make it reusable for both purposes.

[pq]

Cool. I was looking at this recently. I created an issue and will take a look (unless @bwilkerson you already are!)

[bwilkerson]

No, I'm not looking at it.