Bozhidar Batsov
Bozhidar Batsov
I think this should be some configuration option, not a default behavior.
My point is that if a cop is supposed to register redundant parentheses anything that's redundant, but someone might want to keep should be considered some special case. In general...
Also - it seems me it'd be good to extend the example's in the cop's documentation to cover more cases of redundant parentheses (e.g. like the one we're discussing right...
I like the cop, but I'm not sure about the name. Perhaps this cop should consider constants as well (e.g. `METHOD_NAME`).
@koic Can you take a look here?
> In the very, very long term I would love to have a conversation with you all about the feasibility of making it the main parser. Trust me I know...
> With Ruby 3.4 (or Ruby 3.5?), the relationship between Lrama and Prism as a Universal Parser will likely progress further, so until then, it might be better to keep...
> Multi-Version Support: As you said this is planned. I would love to understand the true requirement here for rubocop. Is the actual requirement "be able to parse as any...
@koic I like your idea!
> I do still believe Style/RedundantParentheses can be useful for catching parenthesis around singular logical statements (ex: if (variable.nil?)). Those are super rare in practice, though, so providing special support...