search

idearium / eslint-config-idearium

Idearium ESLint config.
MIT License
0 stars 1 forks source link

Set arrow-parens to always required #10

Closed smebberson closed 6 years ago

smebberson commented 6 years ago

It's a real pain in the butt to add and remove () as you add, change and remove parameters to arrow functions. Just to simplify things, I think () should always be required. Less mucking around.

This PR makes that change.

smebberson commented 6 years ago

@allanchau, thoughts?

allanchau commented 6 years ago

@smebberson Hrmm, personally I prefer less brackets eg smsList.map(sms => sms.sid) because I find it easier to scan over and it looks a bit cleaner. Pretty much the same reason why I don't have the indent guides in my editor. If it is slowing things down for you though, I don't mind changing it.

smebberson commented 6 years ago

Yeah, I prefer less brackets, but you need brackets in most situations, 3/4. There is only one situation that doesn't require them. Don't you find it annoying and time consuming to put them in and remove them all the time?