Skip to content
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

feat: add generate_mut_trait_impl assist #15832

Merged
merged 2 commits into from
Nov 5, 2023

Conversation

Young-Flash
Copy link
Member

generate_mut_trait_impl

Generate proper index_mut method body refer to index method body may impossible due to the unpredicable case (#15581).

Here just leave the index_mut method body be same as index method body, user can modify it manually to meet their need.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 4, 2023
@lnicola
Copy link
Member

lnicola commented Nov 4, 2023

r? @alibektas

Copy link
Member

@alibektas alibektas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am happy with the current version of things, but let me say that I can't really be trusted in the review business as I am quite new to the project. So I leave the judgment to @lnicola

@lnicola
Copy link
Member

lnicola commented Nov 5, 2023

@bors r+

Thanks, Young-Flash and alibektas!

@bors
Copy link
Contributor

bors commented Nov 5, 2023

📌 Commit b84940b has been approved by lnicola

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Nov 5, 2023

⌛ Testing commit b84940b with merge c1c9e10...

@bors
Copy link
Contributor

bors commented Nov 5, 2023

☀️ Test successful - checks-actions
Approved by: lnicola
Pushing c1c9e10 to master...

@bors bors merged commit c1c9e10 into rust-lang:master Nov 5, 2023
9 checks passed
@Young-Flash Young-Flash deleted the generate_mut_trait branch November 6, 2023 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants