Enable Style/IfWithBooleanLiteralBranches

[nvasilevski]

I find it really hard to find a reason for this rule to be disabled. It is very unlikely that in any situation a code like
is_admin = current_user.is_admin? ? true : false will be more readable than is_admin = is_admin

And even when the result is flipped like is_admin = current_user.ordinary_user? ? false : true an alternative - is_admin = !current_user.ordinary_user? looks much more readable to me

To be honest I even disagree that we should have anything in AllowedMethods since I find !!value.nonzero? to be much more readable than value.nonzero? ? true : false

[rafaelfranca]

I run this cop on Shopify/shopify and I saw no valid use case for using the disallowed style

[sambostock]

I've been meaning to go and audit default cops we've disabled, and either enable them, or add a comment explaining why we've chosen to disable them.