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

[Feature Request] Method Signature Refactoring #2104

Closed
DreamFulFil opened this issue Sep 9, 2021 · 6 comments · Fixed by #2967
Closed

[Feature Request] Method Signature Refactoring #2104

DreamFulFil opened this issue Sep 9, 2021 · 6 comments · Fixed by #2967

Comments

@DreamFulFil
Copy link

All prevalent IDEs such as Eclipse, IntelliJ, Netbeans provides a way to refactor method signatures.
It would be nice to have such a feature implemented in VSCode since the methods are usually spread across the entire project, and currently no such refactoring actions are supported(checked the documentation and opened/closed issues).
Visual Studio seems to have such an existing feature

Thank you for the hard work.

@rgrunber
Copy link
Member

rgrunber commented Sep 10, 2021

It should be possible, but the tricky part is creating a UI around it. We could probably have a look at https://git.eclipse.org/c/jdt/eclipse.jdt.ui.git/tree/org.eclipse.jdt.ui/ui%20refactoring/org/eclipse/jdt/internal/ui/refactoring/ChangeSignatureWizard.java as a start point.

@Med-H
Copy link

Med-H commented Feb 12, 2022

any news on this?

@coltongoates
Copy link

I'm also anxiously awaiting this feature. How is feature work prioritized? Can we vote somewhere?

@ewot
Copy link

ewot commented Oct 27, 2022

This is now implemented in the Netbeans language server plugin: https://marketplace.visualstudio.com/items?itemName=ASF.apache-netbeans-java

That plugins has other issues however, so I hope we will get this soon.

@CsCherrYY
Copy link
Contributor

I'll take a look these days.

@CsCherrYY
Copy link
Contributor

we have an upstream PR: eclipse-jdt/eclipse.jdt.ui#428 which will make the processors available in jdt.ls.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

7 participants