Skip to content

Sort imports for dartfmt #20244

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
DartBot opened this issue Jul 29, 2014 · 4 comments
Closed

Sort imports for dartfmt #20244

DartBot opened this issue Jul 29, 2014 · 4 comments
Assignees
Labels
legacy-area-analyzer Use area-devexp instead. type-enhancement A request for a change that isn't a bug

Comments

@DartBot
Copy link

DartBot commented Jul 29, 2014

This issue was originally filed by [email protected]


What steps will clearly show the issue / need for enhancement?

  1. Run dartfmt -t on attached foo.dart
  2. Observe that imports are not in order.

What is the current output?
`
import 'package:a_star/a_star_2d.dart';
import 'package:zx/zx.dart';
import 'dart:convert';
import 'dart:async';
import 'package:a_star/a_star.dart';

void main() {
}
`

What would you like to see instead?
`
import 'dart:convert';
import 'dart:async';
import 'package:a_star/a_star.dart';
import 'package:a_star/a_star_2d.dart';
import 'package:zx/zx.dart';

void main() {
}
`

What version of the product are you using? On what operating system?
Dart VM version: 1.5.3 (Thu Jul 3 02:27:23 2014) on "macos_x64"

Please provide any additional information below.


Attachment:
foo.dart (166 Bytes)

@DartBot
Copy link
Author

DartBot commented Jul 29, 2014

This comment was originally written by [email protected]


Correction on "What would you like to see":
`
import 'dart:async';
import 'dart:convert';
import 'package:a_star/a_star.dart';
import 'package:a_star/a_star_2d.dart';
import 'package:zx/zx.dart';

void main() {
}
`

@iposva-google
Copy link
Contributor

Added Area-Formatter, Triaged labels.

@munificent
Copy link
Member

Phil, this is a linter bug. Want to do something with it?


Set owner to @pq.
Removed Area-Formatter label.
Added Area-Analyzer label.

@pq
Copy link
Member

pq commented Feb 13, 2015

Indeed, yes. Thanks Bob.

Moved here: #57172


Added MovedToGithub label.

@kevmoo kevmoo added type-enhancement A request for a change that isn't a bug and removed priority-unassigned labels Mar 1, 2016
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
legacy-area-analyzer Use area-devexp instead. type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

5 participants