Clarification of bracketSpacing option

I’m not 100% sure what the bracketSpacing config does.

These explanations:

• // Controls the printing of spaces inside object literals from the README.md
• // Controls the printing of spaces inside arrays and objects from the atom package settings page

When using, the following code:

// source:
let list = [{
    name: "foo",
    value: "bar"
  },{
    name: "bar",
    value: "baz"
  }];

The output is always the same, wether bracketSpacing is used or not (cli or atom plugin):

// output:
let list = [
  {
    name: "foo",
    value: "bar"
  },
  {
    name: "bar",
    value: "baz"
  }
];

I’m assuming I’m just expecting something its not designed to do. With bracketSpacing disabled, I would have expected it to stay more dense. Any clarification would be great. Thanks!

[jlongster]

People are pretty opinionated about trailing commas, and I think it’s worth having it as an option. However, I am open to removing the bracketSpacing option. I haven’t seen many people talk about it or use it. I think it’s mainly there because recast had that option.

[vjeux]

Making trailing commas the default seems like a good idea! @jlongster we could use that option for the es7 trailing commas. I don’t think that trailing commas are very controversial