[New Feature]: Support for renaming methods coming from type inclusions #38481
Labels
Area/Rename
Language Server Rename related issues
IceBox
Older issues that are not being actively worked on but may be revisited in the future.
Points/4
Equivalent to four day effort
Priority/Low
Team/LanguageServer
Language Server Implementation related issues. #Compiler
Type/NewFeature
Description
This is more of a suggestion. Need to consider and see if what I'm proposing is feasible/makes sense from a language semantics POV. Consider the following:
Currently when we rename
foo()
method at either <1> or <2>, it only renames that particular case (along with its usages I presume). Suppose we renamefoo
tobar
at <1>. Then this breaks all the places in the code where we have usedFoo
as a type inclusion. So can we consider or rather is it correct to considerfoo()
at <2> a reference offoo()
at <1> and renamefoo
tobar
at <2> as well? The inverse shouldn't happen though: renamingfoo()
at <2> tobar()
shouldn't changefoo()
at <1>.Describe your problem(s)
No response
Describe your solution(s)
No response
Related area
-> Editor
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response
The text was updated successfully, but these errors were encountered: