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 method with property refactoring sometimes eats new line #61161

Closed
DoctorKrolic opened this issue May 6, 2022 · 1 comment · Fixed by #75027
Closed

Replace method with property refactoring sometimes eats new line #61161

DoctorKrolic opened this issue May 6, 2022 · 1 comment · Fixed by #75027
Labels
Area-IDE Bug help wanted The issue is "up for grabs" - add a comment if you are interested in working on it IDE-CodeStyle Built-in analyzers, fixes, and refactorings
Milestone

Comments

@DoctorKrolic
Copy link
Contributor

Ported from #40758 (comment)

Steps to Reproduce:

class C
{
    public int Test1() { return 1; }
    public int Test2() { return 0; }
}

Execute Replace method with property on Test1

Expected Behavior:

class C
{
    public int Test1 => 1;
    public int Test2() { return 0; }
}

Actual Behavior:

class C
{
    public int Test1 => 1; public int Test2() { return 0; }
}
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead labels May 6, 2022
@vatsalyaagrawal vatsalyaagrawal added Bug help wanted The issue is "up for grabs" - add a comment if you are interested in working on it and removed untriaged Issues and PRs which have not yet been triaged by a lead labels May 13, 2022
@vatsalyaagrawal vatsalyaagrawal added the IDE-CodeStyle Built-in analyzers, fixes, and refactorings label May 13, 2022
@vatsalyaagrawal vatsalyaagrawal added this to the Backlog milestone May 13, 2022
@Orachor
Copy link

Orachor commented Sep 5, 2024

I'll take a look into this :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-IDE Bug help wanted The issue is "up for grabs" - add a comment if you are interested in working on it IDE-CodeStyle Built-in analyzers, fixes, and refactorings
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants