Skip to content

Commit 54c2ee9

Browse files
committed
Auto merge of rust-lang#15219 - alibektas:15080, r=Veykril
Unify getter and setter assists This PR combines what previously have been two different files into a single file. I want to talk about the reasons why I did this. The issue that prompted this PR ( and before I forget : this pr fixes rust-lang#15080 ) mentions an interesting behavior. We combine these two assists into an assist group and the order in which the assists are listed in this group changes depending on the text range of the selected area. The reason for that is that VSCode prioritizes actions that have a bigger impact in a smaller area and until now generate setter assist was only possible to be invoked for a single field whereas you could generate multiple getters for the getter assist. So I used the latter's infra to make former applicable to multiple fields, hence the unification. So this PR solves in essence 1. Make `generate setter` applicable to multiple fields 2. Provide a consistent order of the said assists in listing.
2 parents db17f79 + 1829d55 commit 54c2ee9

File tree

4 files changed

+324
-300
lines changed

4 files changed

+324
-300
lines changed

0 commit comments

Comments
 (0)