New rule: no bitwise operators on non integers

[tom-sherman]

I came across an error in my code where I typed 'string' | b instead of 'string' || b.

These bitwise operators only make sense on integers and a rule to catch this would have saved me a bit of time!

What are thoughts around a rule that disallows any string or object literal symbols in these scenarios?

[LinusU]

Sounds very good!

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

[feross]

Reposting the comment I wrote on the PR for this issue:

@tom-sherman I appreciate the idea behind this PR, but I don't think we should disallow bitwise operators when used with numbers or in the case that @LinusU mentioned:

fs.access(path.join(TMP, name), fs.R_OK | fs.W_OK, function (err) {

If you can add an exception to ESLint that scopes this rule sufficiently down to the cases we actually want to target, then I'm happy to consider this change! In the meantime, I'm going to close this issue since we can't enable this rule without an upstream change from ESLint. Please post a comment here and I will re-open this issue, once there's an issue or PR filed with ESLint.