-
Notifications
You must be signed in to change notification settings - Fork 226
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix S3247 FP: Should not report on member access objects #9486
Conversation
ed038b6
to
cc6a032
Compare
5880ed1
to
ff65b2b
Compare
cc6a032
to
e2340ef
Compare
ff65b2b
to
22c38fe
Compare
e2340ef
to
2e7085c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
_ = (Fruit)f.Property; // Secondary | ||
} | ||
|
||
if(LocalProperty is Fruit) // Noncompliant |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nitpick, missing space:
if(LocalProperty is Fruit) // Noncompliant | |
if (LocalProperty is Fruit) // Noncompliant |
Quality Gate passed for 'Sonar .NET Java Plugin'Issues Measures |
Quality Gate passed for 'SonarAnalyzer for .NET'Issues Measures |
Peach validation revealed FNs: #9491 created. |
Fixes #9485