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 S3415 FP/FN: Support named arguments #6630
Labels
Area: C#
C# rules related issues.
Sprint: Hardening
Short-lived label (that will live long due to reusability)
Type: False Negative
Rule is NOT triggered when it should be.
Type: False Positive
Rule IS triggered when it shouldn't be.
Projects
Milestone
Comments
cristian-ambrosini-sonarsource
added a commit
that referenced
this issue
Jan 13, 2023
cristian-ambrosini-sonarsource
added a commit
that referenced
this issue
Jan 16, 2023
cristian-ambrosini-sonarsource
added a commit
that referenced
this issue
Jan 17, 2023
cristian-ambrosini-sonarsource
added a commit
that referenced
this issue
Jan 23, 2023
martin-strecker-sonarsource
pushed a commit
that referenced
this issue
Jan 23, 2023
The issue should be solved by using sonar-dotnet/analyzers/src/SonarAnalyzer.Common/Helpers/MethodParameterLookupBase.cs Line 23 in 49f172c
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Area: C#
C# rules related issues.
Sprint: Hardening
Short-lived label (that will live long due to reusability)
Type: False Negative
Rule is NOT triggered when it should be.
Type: False Positive
Rule IS triggered when it shouldn't be.
Description
S3415 doesn't support named arguments and considers only the positions of those, resulting in the following FN/FP:
The rule only considers literals but constant values should be checked as well:
The text was updated successfully, but these errors were encountered: