dotnet/roslyn

Get naming warnings on parameters when implementing indexed properties

Open

#41,531 opened on Feb 10, 2020

View on GitHub
 (0 comments) (0 reactions) (0 assignees)C# (4,257 forks)batch import
Area-IDEBugIDE-CodeStylehelp wanted

Repository metrics

Stars
 (20,414 stars)
PR merge metrics
 (Avg merge 6d 17h) (256 merged PRs in 30d)

Description

Version Used:

I think this is occurring due to the indexed property, this particular case came from: https://docs.microsoft.com/en-us/dotnet/api/vslangproj.projectproperties.extender?view=visualstudiosdk-2017.

Steps to Reproduce:


    Public Class ProjectProperties

        Public Property Extender(ByVal ExtenderName As String) As Object
            Get
                Return Nothing
            End Get
        End Property    

    End Class

 
    public partial class VSProject : VSLangProj.ProjectProperties
    {
        public object get_Extender(string ExtenderName) => throw new NotImplementedException();
    }

Expected Behavior: No warning

Actual Behavior:

Severity	Code	Description	Project	File	Line	Suppression State
Error	IDE1006	Naming rule violation: The first word, 'ExtenderName', must begin with a lower case character	Microsoft.VisualStudio.ProjectSystem.Managed.VS	C:\project-system\src\Microsoft.VisualStudio.ProjectSystem.Managed.VS\ProjectSystem\VS\Automation\VsProject_VsLangProjectProperties.cs	181	Active

Contributor guide