Hugo Vacher
Hugo Vacher
I ended up trying to make that new rule as consistent and generic as possible. The main non-starter for us, that means Shopify is [not using the `*LineBreak(s)` cops as...
I was curious to see how **ESLint** handles it: looks like in most cases there is no option it's just "on" by default if you allow single line calls: For...
I think the ESLint behaviour would be perfect for us actually, either through an option, or by default, it doesn't have to be as flexible or generic as my pull...
In the meantime I took the advice and renamed the option, it's a much better name! > Also the name of the flag can be better - e.g. `AllowMultilineFinalElement` or...
> I'm wondering if this shouldn't be more or less a single toggle that all cops should respect. 😮 is that a pattern Rubocop uses? That does definitely sounds better...
The main downside with a global rule, is that people lose the flexibility to allow this only in certain constructs, I could see how some folks would prefer to allow...
Hey folks. Would love some more input on this! I'm happy to work on a more [opinionated/restricted PR](https://github.com/rubocop/rubocop/pull/10812#issuecomment-1184452963) too if that's how we prefer it! Or a global rule too!
@bbatsov so what's the decision on this? 😄
@bbatsov any chance we could get this moving? I'm happy to explore alternative approaches, like a global option, or a more limited, opinionated approach, just need your input on what...