Trailing comma (was: sorting with unexpected results after update) about import-sort

[bruno12mota]

Oh nice, thanks 😄 and sorry didn't check that issue

Trailing comma is a bummer but not totally critical for me. Would be nice to have as an option though

[renke]

Hi @bruno12mota

thanks for the kind words. Yes. you are right, import-sort was updated to a new major version and a few things changed. Have a look at renke/atom-import-sort#10 which explains how to get back the old style.

However, trailing comma is currently not configurable 😢, sorry.

[bruno12mota]

Also, multiline imports are appending a trailing comma to the end.

Initial

import {
  GraphQLBoolean,
  GraphQLID,
  GraphQLNonNull,
  GraphQLString
} from 'graphql';

After sorting

import {
  GraphQLBoolean,
  GraphQLID,
  GraphQLNonNull,
  GraphQLString,
} from 'graphql';

Which wasn't happening before as well

[renke]

Hi @bruno12mota

a trailing comma is now only added if it was put there in the first place.