dotnet/roslyn

IDE0040 'Accessibility modifiers required' should be configurable to ignore partial containing classes

Aperta

#52.904 aperta il 25 apr 2021

 (12 commenti) (9 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-IDEConcept-Continuous ImprovementFeature - IDE0040help wanted

Metriche repository

Star
 (20.414 stelle)
Metriche merge PR
 (Merge medio 6g 17h) (256 PR mergiate in 30 g)

Descrizione

I don't like duplicating the accessibility modifiers on the second file, but out of the box now, .NET 5 puts a suggestion annotation in the code and the error list to do so:

internal sealed partial class A // Or some other modifiers
{
    // ...
}
// ℹ IDE0040 Accessibility modifiers required
//            ↓
partial class A
{
    private sealed class B
    {
        // ...
    }
}

Could IDE0040 please skip analyzing classes that are only specifying the minimal amount necessary to declare the nested type? If not, could this be made configurable?

Guida contributor