-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
ALEFileRename command added. #4012
Conversation
This command renames file and uses tsserver `getEditsForFileRename` to fix import paths in Typescript files.
@hsanson please review, I'm not sure who else could check this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not have the setup to test this on my machine but looks good to me. It only needs some polishing for when someone runs ALERename when another LSP is running.
For example running ALERename with pyright LSP active shows "E488: Trailing characters" error.
Also running ALERename with no arguments results in "E119: Not enough arguments for function: ale#util#Input" error. Would be better to detect the command is being wrongly used and show a more user friendly error message.
I think it is good now. Tested with pyright - no problem. It seems all the problems were related to ale#util#Input change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution and changes.
🍻 |
This command renames file and uses tsserver
getEditsForFileRename
tofix import paths in Typescript files.