issues Search Results · repo:SonarSource/sonar-dotnet language:C#
Filter by
3k results
(92 ms)3k results
inSonarSource/sonar-dotnet (press backspace or delete to remove)When there s a Test namespace based on project name with .Test in it, and follow up namespace Something, and there s a
using importing the Something directly, raise an issue to use .Test suffix instead. ...
Type: Styling
pavel-mikula-sonarsource
- Opened 23 days ago
- #9732
interface IInterface
{
void InterfaceMethod();
}
class Base
{
protected virtual void BaseMethod() {}
}
class Sample : Base, IInterface
{
public async void InterfaceMethod() {} // ...
Area: C#
Type: Rule Idea
Tim-Pohlmann
- 3
- Opened on Nov 29, 2024
- #9711
Description
In an async test method, S6966 is raised for Moq s Mock.Raise method, suggesting to use RaiseAsync instead.
RaiseAsync does not work for normal events and requires a Task-returning event ...
Area: C#
Type: False Positive
a10r
- 2
- Opened on Nov 25, 2024
- #9697
Description
Async implementations of non-async interface methods are not properly reported as an issue.
Repro steps
Minimal reproduction code:
public interface ITestReproduction
{
void TestMethod(); ...
qhris
- 4
- Opened on Nov 22, 2024
- #9696
Description
XAML event handlers cannot be static. Currently S2325 is raised on XAML event handlers.
The issue is the same as #5536, only instead of WinForms it applies to XAML.
Repro steps
private ...
Area: C#
Type: False Positive
PetrJasek89
- 3
- Opened on Nov 18, 2024
- #9695
Description
Rule ID: S3257 UserControl provides the method AddHandler(RoutedEvent routedEvent, Delegate handler). If I pass new
RoutedEventHandler(OnErrorEvent) as the handler argument the rule is raised. ...
lsw-sma
- 1
- Opened on Nov 12, 2024
- #9694
Description
Rule Id: S6966 Microsoft.data.SqlClient currently offer Async methods for almost all their API, but these Async methods
suffer from hidden performance hit. The rule should not raise for the ...
LudovicT
- 2
- Opened on Nov 6, 2024
- #9693
Description
S1144 is being raised unexpectedly for template-method implementations.
Repro steps
public class Repro()
{
var x = new SomeClass();
const string theName = x.Name;
var y = x.DeepClone(); ...
Xilconic
- 5
- Opened on Nov 3, 2024
- #9692
When the Nullable context is disabled, the nullable forgiving operator is a no-op. There s no need to use it.
#nullable disable
string s = Method();
_ = s!.ToString(); // Noncompliant, as it s noop ...
Area: C#
Type: Rule Idea
mary-georgiou-sonarsource
- Opened on Oct 31, 2024
- #9691
Description
S6964 complains about under-posting when properties are not nullable and annotated with BindRequiredAttribute.
Indicates that a property is required for model binding. When applied to a property, ...
Area: C#
Type: False Positive
PaulBol
- 1
- Opened on Oct 30, 2024
- #9690
Learn how you can use GitHub Issues to plan and track your work.
Save views for sprints, backlogs, teams, or releases. Rank, sort, and filter issues to suit the occasion. The possibilities are endless.Learn more about GitHub IssuesProTip!
Restrict your search to the title by using the in:title qualifier.Learn how you can use GitHub Issues to plan and track your work.
Save views for sprints, backlogs, teams, or releases. Rank, sort, and filter issues to suit the occasion. The possibilities are endless.Learn more about GitHub IssuesProTip!
Restrict your search to the title by using the in:title qualifier.