SA1119 incorrectly reporting on pattern matching (is expression)

[nbarbettini]

Given this code:

object obj = null;

if (!(obj is string foo))
{
    // do something with foo
}

SA1119 reports the parenthesis as unnecessary (they are necessary). The code fix removes the parenthesis, resulting in syntax that won't compile:

if (! obj is string foo)
{
    // do something with foo
}

It would appear SA1119 is reaching for the proposed is not or aint operator. 😉

[nbarbettini]

This should be part of #2268 since it's caused by a new C# 7 feature (is-expression pattern matching).

[sharwell]

This particular pattern should already be covered by the light-up functionality in #2370, allowing someone to branch off that work and correct this issue.

[AdjutantML]

Thank you for fixing this