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
Support auto-import in macro #4268
Conversation
&import, | ||
edit.text_edit_builder(), | ||
); | ||
edit.target(range); |
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 can't get the AssistContext
in this closure, so I pass the ctx
to ActionBuilder
. Another option is passing the ctx
directly as closure arguments, but I don't want to change too much code. I will prefer to refactor it in other PR.
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'm currently trying to implement #1062 and looks like I will have to change it in order to use the insert_use_statement
in ra_ide
.
So give it a couple of weeks before doing the refactoring :)
bors r+ |
🔒 Permission denied Existing reviewers: click here to make SomeoneToIgnore a reviewer |
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.
Well, so much for a try.
But I think it's fine to merge this as is.
bors r=SomeoneToIgnore
huh, I thought you have r+ rights? Re-added to bors anyway.
…On Mon, 4 May 2020 at 00:33, Kirill Bulatov ***@***.***> wrote:
***@***.**** approved this pull request.
Well, so much for a try.
But I think it's fine to merge this as is.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#4268 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AANB3M56E4VYYF63GESGE5TRPXWKTANCNFSM4MXZZDBA>
.
|
Build succeeded: |
Fixed: #3854