-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
internal: Migrate assists to the structured snippet API, part 7/7 #16577
internal: Migrate assists to the structured snippet API, part 7/7 #16577
Conversation
Used for allowing newly generated syntax constructs to be renamed without having to go through a separate rename step.
All assists have been migrated to use the structured snippet versions of these methods.
ae6d4fe
to
4af075d
Compare
@nemethf Discovered another snippet rendering issue which causes the placeholders to be placed a little weirdly, but they do still get placed and end up in the final |
#16579 fixes the snippet rendering issues I encountered while testing this. |
Thanks! |
☀️ Test successful - checks-actions |
Continuing from #16467
Migrates the following assists:
generate_trait_from_impl
This adds
add_placeholder_snippet_group
, which adds a group of placeholder snippets which are linked together and allows for renaming generated items without going through a separate rename step.This also removes the last usages of
SourceChangeBuilder::{insert,replace}_snippet
, as all assists have finally been migrated to the structured snippet versions of those methods.