feat: add object-curly-newline rule

[Talent30]

@sindresorhus close xojs/xo#367

[sindresorhus]

What's the reasoning?

[Talent30]

Thank you for asking! Upon reflection, I believe this might not be a good idea.

let {
g, h
} = obj;

The above format isn't very good IMO.

I am leaning towards to set this option as never now until the community enhance the object-property-newline rule or add destructuring-property-newline.

Happy to hear your thoughts :)

[Talent30]

If we add object-property-newline as well we can achieve the following result

Incorrect

let a = {foo: 1
};
let b = {
    foo: 1};
let c = {foo: 1, bar: 2
};

Object.assign(stream, {isTTY: true, columns, rows});

Correct

Object.assign(stream, {
    isTTY: true,
    columns,
    rows
});


let a = {foo: 1};
let b = {foo: 1};
let c = {
    foo: 1,
    bar: 2
};

[Talent30]

Thank you very much for your feedback

[voxpelli]

For future reference, this doesn't seem to fully work as desired yet, as it now (as noted above) auto-fixes { foo, bar } into:

{
  foo, bar
}

See eg: sindresorhus/type-fest#838 (review)