aboutsummaryrefslogtreecommitdiff
path: root/crates/ra_assists
Commit message (Collapse)AuthorAgeFilesLines
...
* Minor cleanupsAleksey Kladov2020-05-052-6/+9
|
* Fix compilationAleksey Kladov2020-05-051-1/+1
|
* Move group_label where it belongsAleksey Kladov2020-05-052-7/+6
|
* use GroupLabel for type safetyAleksey Kladov2020-05-051-3/+4
|
* Minor renameAleksey Kladov2020-05-051-3/+3
|
* Flip Assist::new argumentsAleksey Kladov2020-05-052-4/+4
|
* Rename ImplItem to AssocItemEdwin Cheng2020-05-053-20/+20
|
* Merge #4265bors[bot]2020-05-051-3/+10
|\ | | | | | | | | | | | | | | 4265: Make `change_visibility` assist work for tuple struct fields r=matklad a=Jesse-Bakker Fixes #4206 Co-authored-by: Jesse Bakker <[email protected]>
| * Make `change_visibility` assist work for tuple struct field visibilityJesse Bakker2020-05-021-3/+10
| |
* | SimplifyAleksey Kladov2020-05-041-33/+34
| |
* | SimplifyAleksey Kladov2020-05-041-13/+13
| |
* | Support auto-import in macroEdwin Cheng2020-05-025-19/+66
|/
* Add unwrap block assist #4156Benjamin Coenen2020-05-021-3/+2
| | | | Signed-off-by: Benjamin Coenen <[email protected]>
* Merge branch 'master' of github.com:rust-analyzer/rust-analyzerBenjamin Coenen2020-05-028-91/+134
|\
| * Add missing members generates indented blocksAleksey Kladov2020-05-022-79/+119
| |
| * Introduce EffectExprAleksey Kladov2020-05-024-11/+12
| |
| * Remove lower_path from AssistCtx to SemanticEdwin Cheng2020-05-012-7/+4
| |
| * Introduce LowerCtx for path loweringEdwin Cheng2020-05-013-4/+9
| |
* | Add unwrap block assist #4156Benjamin Coenen2020-05-021-76/+13
| | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]>
* | Add unwrap block assist #4156Benjamin Coenen2020-05-011-46/+23
| | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]>
* | Merge branch 'master' of github.com:rust-analyzer/rust-analyzerBenjamin Coenen2020-05-016-90/+237
|\|
| * More principled approach for finding From traitAleksey Kladov2020-04-293-37/+94
| |
| * Use specific pattern when translating if-let-else to matchAleksey Kladov2020-04-294-20/+123
| | | | | | | | | | | | We *probably* should actually use the same machinery here, as we do for fill match arms, but just special-casing options and results seems to be a good first step.
| * Move shared assist code to utilsAleksey Kladov2020-04-293-48/+35
| |
* | Add unwrap block assist #4156Benjamin Coenen2020-04-292-3/+3
| | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]>
* | Add unwrap block assist #4156Benjamin Coenen2020-04-293-0/+456
|/ | | | Signed-off-by: Benjamin Coenen <[email protected]>
* Switch to TryFromAleksey Kladov2020-04-254-6/+6
|
* Convert tests to text-sizeAleksey Kladov2020-04-251-2/+1
|
* Convert code to text-sizeAleksey Kladov2020-04-2521-56/+56
|
* Add `pub(crate)` to functions generated in other moduleTimo Freiberg2020-04-211-46/+23
|
* Jump to sourceChanges in other filesTimo Freiberg2020-04-212-13/+10
|
* Add target file information to AssistActionTimo Freiberg2020-04-213-16/+78
|
* Make add_function generate functions in other modules via qualified pathTimo Freiberg2020-04-212-26/+197
|
* Do not add default and closure types in 'add explicit type' assistKirill Bulatov2020-04-211-5/+43
|
* Fix panic in split_imports assistAleksey Kladov2020-04-201-1/+6
| | | | | | | | | | | | | | | The fix is admittedly quit literally just papering over. Long-term, I see two more principled approaches: * we switch to a fully tree-based impl, without parse . to_string step; with this approach, there shouldn't be any panics. The results might be nonsensical, but so was the original input. * we preserve the invariant that re-parsing constructed node is an identity, and make all the `make_xxx` method return an `Option`. closes #4044
* Some clippy fixesJeremy Kolb2020-04-193-3/+3
|
* Change add_function assist to use todo!()Timo Freiberg2020-04-132-28/+28
|
* Fix PRIceSentry2020-04-131-14/+8
|
* Generalize test and clean up importsIceSentry2020-04-131-8/+7
|
* Fix double comma when merge imports on second lineIceSentry2020-04-131-3/+37
| | | | | | | | | | | This fixes the a bug when merging imports from the second line when it already has a comma it would previously insert a comma. There's probably a better way to check for a COMMA. This also ends up with a weird indentation, but rust-fmt can easily deal with it so I'm not sure how to resolve that. Closes #3832
* Remove more unnecessary bracesLaurențiu Nicola2020-04-121-2/+2
|
* Align grammar for record patterns and literalsAleksey Kladov2020-04-111-19/+13
| | | | | | The grammar now looks like this [name_ref :] pat
* Merge #3925bors[bot]2020-04-113-0/+247
|\ | | | | | | | | | | | | | | | | | | 3925: Implement assist "Reorder field names" r=matklad a=geoffreycopin This PR implements the "Reorder record fields" assist as discussed in issue #3821 . Adding a `RecordFieldPat` variant to the `Pat` enum seemed like the easiest way to handle the `RecordPat` children as a single sequence of elements, maybe there is a better way ? Co-authored-by: Geoffrey Copin <[email protected]>
| * Generate docGeoffrey Copin2020-04-111-0/+15
| |
| * Fix doc testsGeoffrey Copin2020-04-111-2/+1
| |
| * Add documentation commentGeoffrey Copin2020-04-111-0/+16
| |
| * Avoid adding a RecordFieldPat variant to the Pat enumGeoffrey Copin2020-04-111-39/+47
| |
| * Remove Option unwrapingGeoffrey Copin2020-04-111-3/+3
| |
| * Implement assist "Reorder field names"Geoffrey Copin2020-04-092-0/+209
| |
* | Change missing impl assist to use todo!() instead of unimplemented()Chris Hopman2020-04-102-18/+18
| | | | | | | | | | | | | | | | | | | | | | | | todo!() "Indicates unfinished code" (https://doc.rust-lang.org/std/macro.todo.html) Rust documentation provides further clarification: > The difference between unimplemented! and todo! is that while todo! > conveys an intent of implementing the functionality later and the > message is "not yet implemented", unimplemented! makes no such claims. todo!() seems more appropriate for assists that insert missing impls.