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

EnC: Moving a method with a parameter across partial definitions issues wrong rude edit #52211

Open
davidwengier opened this issue Mar 28, 2021 · 1 comment

Comments

@davidwengier
Copy link
Contributor

Version Used:
Version 16.10.0 Preview 2.0 [31128.4.main]

Steps to Reproduce:

Before edit:

File 1:

partial class C
{
   void F2() {}
}

File 2:

partial class C
{
   void F2(int a) {}
}

After edit:

File 1:

partial class C
{
   void F2(int a) {}
}

File 2:

partial class C
{
   void F2() {}
}

Expected Behavior:

Each document has semantic update of F2, respectively.

Actual Behavior:

"Add parameter" rude edit is reported.

@davidwengier davidwengier added this to the Backlog milestone Mar 28, 2021
@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged Issues and PRs which have not yet been triaged by a lead label Mar 28, 2021
@davidwengier
Copy link
Contributor Author

FYI @tmat, not sure if you want to just consider this another part of #51011

@jinujoseph jinujoseph removed the untriaged Issues and PRs which have not yet been triaged by a lead label Mar 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants