chore: add arrow-body-style rule

[marcosvega91]

What:

Add arrow-body-style rule

Why:

Because we prefer to use braces around arrow function

How:

updating .eslintrc.js

Checklist:

• Tests
• Ready to be merged
  

[smeijer]

OMG this was a bad idea 😱 😬

How is this:

nodes: result.elements.map((x) => {
  return x.target;
})

better than this:?

nodes: result.elements.map((x) => x.target)

Let's not do this. I'm sorry for wasting your time on this 😇

[marcosvega91]

I didn't like too but I thought for another comment that you want this

[smeijer]

Yeah, I know which comment you're referring to. That's why I said, "This was a bad idea" 😬 (referring to my own suggestion to turn it into a PR)

I did find a way that works though, see #205.

Basically, I do favor { } in statements like if (false) { return } (as I burned my ass more than once by leaving them out), but not in arrow functions when they're not required.

#205 is the opposite of this PR. It removes all return statements from arrow functions when they're not required. I think that looks a lot cleaner 😄

[marcosvega91]

Yes I see is not a problem hahahah 😄