Razmo99
Razmo99
Hello, not sure if this should be another post, I think it's relevant to the discussion. I've been wanting this feature for a while and forked the repo and started...
Hey @JustinGrote I've updated the tests to use Async, they should at least run / compile now
I made some test cases for renaming from within a for / each loop and limiting the rename to the scope if the var is defined within the creation of...
> I've started a new branch that refactors things to the LSP native rename functions, which has the main advantages of requiring no client side code, so should "just work"...
> @Razmo99 once I finish things up and get all tests passing I'll have you help me with more scenarios and ASTVisitor rework, there's others who will jump in too....
@microsoft-github-policy-service agree
@JustinGrotes, are you back in action from Europe? just wondering what are next steps are to move forward. Do you want me to implement the opt in mechanism on the...
I didn't know there was a standard prepearRename provider 😅 I made it the way I did as I wanted a separate endpoint to send rename requests to the language...
> I'm closing this PR in favor of #5056 since the scope has drastically changed. @Razmo99 you should get co-author credit on the PR, if not we'll make sure it...