fix(linter): fix false positives in no-confusing-non-null-assertion

[goto-bus-stop]

This lint did not check which operator was used, so it would warn you that a! + b looks too much like a !+ b, which doesn't make much sense.

  ! typescript-eslint(no-confusing-non-null-assertion): Confusing combinations of non-null assertion and equal test like "a! + b", which looks very similar to not equal "a !+ b".
     ,-[util.ts:115:19]
 114 |     coordEach(wrapped, (coord) => {
 115 |       coord[0] = (coord[0]! + 360) % 360
     :                   ^^^^^^^^^^^^^^^
 116 |     })
     `----
  help: Remove the "!", or prefix the "=" with it.

This patch adds tests for various binary operators and restricts the lint to only those that actually look like !=.

[graphite-app]

Your org has enabled the Graphite merge queue for merging into main

Add the label “merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[Boshen]

I recognize your OG name! Thank you for contributing!

[codspeed-hq]

CodSpeed Performance Report

Merging #4665 will not alter performance

_{Comparing goto-bus-stop:no-confusing-no-confusing-non-null-assertion (853d76e) with main (cbf08d2)}

Summary

✅ 32 untouched benchmarks