Auto sort import conflicts with tslint

[rrousselGit]

I've seen the related issue:

• Organize Imports should used case-insenstive sort for imports #23366

and tested it on the current typescript next (3.1.0-dev.20180825). But the current fix seems to be incomplete.

Code

import { helloWorld, HELLO_WORLD } from 'foo';

Expected behavior:

Using "Organize imports", this should become

import { HELLO_WORLD, helloWorld } from 'foo';

due to the _ in the naming. As this is the expected behavior of tslint ordered-imports

Actual behavior:

Imports are not reordered.
Worse, if you have the code following code which fits tslint rules:

import {  HELLO_WORLD, helloWorld } from 'foo';

then "Organize imports" will revert it back

---

For now I added a // tslint:disable-next-line:ordered-imports but this is not ideal.

[j-oliveras]

Duplicated of #25114?

[rrousselGit]

Oh, didn't see that one. Gotcha