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 more assists to use the structured snippet API #15231
internal: Migrate more assists to use the structured snippet API #15231
Conversation
Can actually split out adding the functions from getting the impl to update or create thanks to being able to refer to the impl ast node. FIXME Context: Unfortunately we can't adjust the indentation of the newly added function inside of `ast::AssocItemList::add_item` since for some reason the `todo!()` placeholder generated by `add_missing_impl_members` and `replace_derive_with_manual_impl` gets indented weirdly.
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.
One super pedantic nit, otherwise looks awesome!
Thanks! @bors r+ |
☀️ Test successful - checks-actions |
1 similar comment
☀️ Test successful - checks-actions |
👀 Test was successful, but fast-forwarding failed: 422 Changes must be made through a pull request. |
internal: Migrate assists to the structured snippet API, part 3 Continuing from #15231 Migrates the following assists: - `add_missing_match_arms` - `fix_visibility` - `promote_local_to_const` The `add_missing_match_arms` changes are best reviewed commit-by-commit since they're relatively big changes compared to the rest of the commits.
Continuing from #14979
Migrates the following assists:
generate_derive
wrap_return_type_in_result
generate_delegate_methods
As a bonus,
generate_delegate_methods
now generates the function and impl block at the correct indentation 🎉.