Skip to content
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

Replace 'Property' with method erases DIM #75999

Closed
RenderMichael opened this issue Nov 20, 2024 · 1 comment · Fixed by #76186
Closed

Replace 'Property' with method erases DIM #75999

RenderMichael opened this issue Nov 20, 2024 · 1 comment · Fixed by #76186
Assignees
Labels
Area-IDE help wanted The issue is "up for grabs" - add a comment if you are interested in working on it
Milestone

Comments

@RenderMichael
Copy link
Contributor

Version Used:

.NET SDK 9.0.100

Steps to Reproduce:
Existing code:

interface I
{
    public string Name
    {
        get
        {
            return string.Empty;
        }
    }
}

After fix:

interface I
{
    string GetName();
}

Diagnostic Id:

IDE did not present one, so I'm unsure whether this is the right repo, or if this is implemented in roslyn-analyzers

Expected Behavior:

interface I
{
    public string GetName()
    {
        return string.Empty;
    }
}

Actual Behavior:

interface I
{
    string GetName();
}
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Compilers untriaged Issues and PRs which have not yet been triaged by a lead labels Nov 20, 2024
@CyrusNajmabadi
Copy link
Member

This is the correct repo for this.

@CyrusNajmabadi CyrusNajmabadi added help wanted The issue is "up for grabs" - add a comment if you are interested in working on it Area-IDE and removed Area-Compilers untriaged Issues and PRs which have not yet been triaged by a lead labels Nov 21, 2024
@CyrusNajmabadi CyrusNajmabadi added this to the Backlog milestone Nov 21, 2024
@github-project-automation github-project-automation bot moved this to InQueue in Small Fixes Nov 21, 2024
@CyrusNajmabadi CyrusNajmabadi self-assigned this Nov 21, 2024
@github-project-automation github-project-automation bot moved this from InQueue to Completed in Small Fixes Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-IDE help wanted The issue is "up for grabs" - add a comment if you are interested in working on it
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

2 participants