Force JSX children expressions to have spacing

[noisysocks]

This change forces us to write:

{ isVisible && (
    <Something
        foo="bar"
    />
) }

Instead of:

{isVisible && (
    <Something
        foo="bar"
    />
)}

It does this by setting the children property on the react/jsx-curly-spacing rule.

To test:

1. Open up e.g. editor/index.js
2. Insert {post} underneath line 88
3. Run npm run lint
4. It should complain that you didn't write { post }

[aduth]

Great 👍

[gziolo]

🎉 should we include Prettier-eslint as a devDependency as the next step? 😎

[ahmadawais]

@gziolo What about the spacing rule issue?

[gziolo]

What about the spacing rule issue?

There is a workaround for it with eslint --fix. @noisysocks confirmed that it works pretty well for him.

[ahmadawais]

@gziolo but then what's the use of Prettier if after running it we'll again run the code through ESLint?

[gziolo]

@gziolo but then what's the use of Prettier if after running it we'll again run the code through ESLint?

It's not the best workflow you can imagine, but still better than format code yourself :)