-
Notifications
You must be signed in to change notification settings - Fork 8
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
Offer rename refactoring of Rascal identifiers #382
Comments
Textedits also cover file renamings, so that's cool if we change a module name |
It would be best if we can reuse information generated by the type-checker; for example the summaries, collect them for all modules in a project and then reverse the references. I would limit the current functionality to within a current project and push cross project uses/renamings to an enhancement issue. We don't even have cross-project incremental type-checking |
Thanks for the directions! |
Feature milestonesMilestone 1 [boilerplate] (#389)
Milestone 2 [locals] (#389)
Milestone 3 [globals] (#397)
Milestone 4 [modules] (#397)
Milestone 5 [grammars]
|
@DavyLandman when will this be available in a new release? |
hi @urbanfly I hope soon, but there are some bugs we're still ironing out for the typechecker. Once those are squashed, our priority will be to make a full release. |
Is your feature request related to a problem? Please describe.
I want to have rename refactoring work for rascal.
When I as a user trigger rename on:
I want rascal to calculate the correct rename operation based on whats currently open in the workspace.
Describe the solution you'd like
LSP has a dedicated api for rename. To pace our development, I would like to have support for rename first, and to the generic refactorings in a later stage. Similarly, if needed, we can split the support for rascal vs the support for DSLs.
Ideally the rename command is executed in a similar way as
outline
is done, we have rascal code that is invoked by the rascal-lsp server.Describe alternatives you've considered
Additional context
Good news is that we already have TextEdits in Rascal, which map cleanly on the edits that LSP needs.
The text was updated successfully, but these errors were encountered: