The 'ignoreRHS' option was an unnecessary part of this lint rule; it forced the user to choose between false-negatives (e.g. if(expr && true)) or false positives `return expr && <ReactComponent>`.

The new logic only checks the right hand side of a logical operator if the logical expression is in a conditional context: e.g. it's inside an if block, or it's the left hand side of another logical expression.

# Conflicts:
#	packages/eslint-plugin/src/rules/no-unnecessary-condition.ts
#	packages/eslint-plugin/tests/rules/no-unnecessary-condition.test.ts