Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge #8802 | bors[bot] | 2021-05-12 | 1 | -9/+126 |
|\ | | | | | | | | | | | | | | | | | | | 8802: Keep comments and attrs when extracting struct from enum variant r=Veykril a=DropDemBits Fixes #6730 Still unsure if existing visibilities of fields should be forced to pub (which is what was previously done), or if it's okay to keep it in the extracted struct. Co-authored-by: DropDemBits <[email protected]> | ||||
| * | Keep comments and attrs when extracting struct from enum variant | DropDemBits | 2021-05-11 | 1 | -9/+126 |
| | | |||||
* | | Merge #8800 | bors[bot] | 2021-05-11 | 1 | -16/+34 |
|\ \ | | | | | | | | | | | | | | | | | | | | | | 8800: feat: Make "pull assignments up" assist work in more cases r=Jesse-Bakker a=Jesse-Bakker Fixes #8771 Co-authored-by: Jesse Bakker <[email protected]> | ||||
| * | | Make "pull assignments up" assist work in more cases | Jesse Bakker | 2021-05-11 | 1 | -16/+34 |
| | | | |||||
* | | | Give MergeBehaviour variants better names | Lukas Wirth | 2021-05-10 | 2 | -3/+3 |
| |/ |/| | |||||
* | | simplify | Aleksey Kladov | 2021-05-10 | 1 | -12/+9 |
| | | |||||
* | | internal: introduce `ast::make::ext` module with common shortcuts | Aleksey Kladov | 2021-05-09 | 11 | -70/+48 |
| | | | | | | | | | | | | | | | | | | | | There's a tension between keeping a well-architectured minimal orthogonal set of constructs, and providing convenience functions. Relieve this pressure by introducing an dedicated module for non-orthogonal shortcuts. This is inspired by the django.shortcuts module which serves a similar purpose architecturally. | ||||
* | | internal: fix make API | Aleksey Kladov | 2021-05-09 | 3 | -36/+18 |
| | | |||||
* | | internal: rewrite **Repalce impl Trait** assist to mutable syntax trees | Aleksey Kladov | 2021-05-09 | 1 | -11/+13 |
| | | |||||
* | | cleanups | Aleksey Kladov | 2021-05-09 | 3 | -5/+6 |
| | | |||||
* | | cleanups | Aleksey Kladov | 2021-05-09 | 3 | -15/+23 |
| | | |||||
* | | minor: fix test style | Aleksey Kladov | 2021-05-09 | 1 | -55/+31 |
| | | |||||
* | | Merge #8773 | bors[bot] | 2021-05-08 | 1 | -1/+1 |
|\ \ | |/ |/| | | | | | | | | | | | 8773: fix: Correctly support SelfType when searching for usages r=Veykril a=Veykril Fixes #7443 Co-authored-by: Lukas Wirth <[email protected]> | ||||
| * | Correctly support SelfType when searching for usages | Lukas Wirth | 2021-05-08 | 1 | -1/+1 |
| | | |||||
* | | feat: add "mentoring instructions" test for pull up assist | Aleksey Kladov | 2021-05-08 | 1 | -1/+32 |
| | | |||||
* | | internal: pull_assignment_up uses mutable trees | Aleksey Kladov | 2021-05-08 | 1 | -69/+68 |
| | | |||||
* | | minor: add missing test | Aleksey Kladov | 2021-05-08 | 1 | -16/+36 |
|/ | |||||
* | internal: remove one more usage of SyntaxRewriter | Aleksey Kladov | 2021-05-08 | 1 | -56/+46 |
| | |||||
* | internal: remove one more syntax rewriter | Aleksey Kladov | 2021-05-08 | 2 | -64/+34 |
| | |||||
* | Merge #8711 | bors[bot] | 2021-05-04 | 3 | -14/+326 |
|\ | | | | | | | | | | | | | | | | | | | 8711: Only resolve selected assist r=matklad a=SomeoneToIgnore Part of https://github.com/rust-analyzer/rust-analyzer/issues/8700 Now resolves only the assist that was selected out of the list, while before the whole assist list was resolved despite a single popup selection. Co-authored-by: Kirill Bulatov <[email protected]> | ||||
| * | Index retrieval fix | Kirill Bulatov | 2021-05-03 | 1 | -1/+1 |
| | | |||||
| * | Small fixes | Kirill Bulatov | 2021-05-03 | 1 | -2/+2 |
| | | |||||
| * | Tests added | Kirill Bulatov | 2021-05-03 | 1 | -1/+242 |
| | | |||||
| * | Add docs and use better naming | Kirill Bulatov | 2021-05-03 | 1 | -5/+20 |
| | | |||||
| * | Resolve single assist only | Kirill Bulatov | 2021-05-03 | 1 | -3/+36 |
| | | |||||
| * | Add a way to resolve certain assists | Kirill Bulatov | 2021-05-03 | 3 | -16/+39 |
| | | |||||
* | | internal: remove one more usage of the rewriter | Aleksey Kladov | 2021-05-04 | 2 | -23/+49 |
| | | |||||
* | | Don't mutate the tree while traversing in reorder_impl | Lukas Tobias Wirth | 2021-05-03 | 1 | -4/+36 |
|/ | |||||
* | internal: fix naming polarity | Aleksey Kladov | 2021-04-30 | 2 | -2/+2 |
| | | | | | Type Constructors have *parameters*, when they are substituted with type *arguments*, we have a type. | ||||
* | internal: normalize name | Aleksey Kladov | 2021-04-30 | 1 | -1/+1 |
| | | | | All def types in hir are unsubstituted | ||||
* | Split out merge_imports module from helpers::insert_use | Lukas Wirth | 2021-04-24 | 2 | -5/+2 |
| | |||||
*-. | Merge #8591 #8638 | bors[bot] | 2021-04-23 | 7 | -152/+164 |
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8591: Remove SyntaxRewriter usage in insert_use in favor of mutable syntax trees r=matklad a=Veykril Unfortunately changing `insert_use` to not use `SyntaxRewriter` creates a lot of changes since so much relies on that. But on the other hand this should be the biggest usage of `SyntaxRewriter` I believe. 8638: Remove SyntaxRewriter::from_fn r=Veykril a=Veykril Co-authored-by: Lukas Wirth <[email protected]> | ||||
| | * | Remove SyntaxRewriter::from_fn | Lukas Wirth | 2021-04-23 | 2 | -15/+21 |
| |/ |/| | |||||
* | | Merge #8317 | bors[bot] | 2021-04-23 | 3 | -0/+559 |
|\ \ | | | | | | | | | | | | | | | | | | | | | | 8317: Convert tuple struct to named struct assist r=Veykril a=unexge Closes https://github.com/rust-analyzer/rust-analyzer/issues/8192 Co-authored-by: unexge <[email protected]> | ||||
| * | | Add missing test case for "Convert to named struct" assist | unexge | 2021-04-23 | 1 | -0/+40 |
| | | | |||||
| * | | Stop iterating reference after made an edit in "Convert to named struct" assist | unexge | 2021-04-23 | 1 | -2/+47 |
| | | | |||||
| * | | Move reference editing logic into own function to make error handling more ↵ | unexge | 2021-04-22 | 1 | -68/+61 |
| | | | | | | | | | | | | ease in "Convert to named struct" assist | ||||
| * | | Add multi file test for "Convert to named struct" assist | unexge | 2021-04-21 | 1 | -0/+33 |
| | | | |||||
| * | | Remove `unwrap`s in "Convert to named struct" assist | unexge | 2021-04-21 | 1 | -9/+19 |
| | | | |||||
| * | | Add larger example for "Convert to named struct" assist | unexge | 2021-04-21 | 2 | -8/+60 |
| | | | |||||
| * | | Fix incorrectly replacing method calls in "Convert to named struct" assist | unexge | 2021-04-21 | 1 | -17/+41 |
| | | | |||||
| * | | Add convert tuple struct to named struct assist | unexge | 2021-04-04 | 3 | -0/+362 |
| | | | |||||
* | | | Merge #8620 | bors[bot] | 2021-04-22 | 1 | -14/+22 |
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8620: Remove unnecessary braces for extracted block expression r=Veykril a=brandondong This change addresses the first bullet point of https://github.com/rust-analyzer/rust-analyzer/issues/7839. Specifically, when extracting block expressions, remove the unneeded extra braces inside the generated function. Co-authored-by: Brandon <[email protected]> | ||||
| * | | | Remove unnecessary braces for extracted block expression | Brandon | 2021-04-22 | 1 | -14/+22 |
| | | | | |||||
* | | | | Add support for fill match arms of boolean values | Comonad | 2021-04-21 | 1 | -23/+199 |
| | | | | | | | | | | | | | | | | - Add support for boolean inside tuple | ||||
* | | | | Merge #8602 | bors[bot] | 2021-04-20 | 1 | -28/+45 |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8602: Fix panic in `replace_derive_with_manual_impl` r=jonas-schievink a=jonas-schievink bors r+ Co-authored-by: Jonas Schievink <[email protected]> | ||||
| * | | | | Follow testing style guide | Jonas Schievink | 2021-04-20 | 1 | -28/+28 |
| | | | | | |||||
| * | | | | Fix panic in `replace_derive_with_manual_impl` | Jonas Schievink | 2021-04-20 | 1 | -0/+17 |
| |/ / / | |||||
* / / / | "Inline variable" when on a use of the variable | Jonas Schievink | 2021-04-20 | 1 | -38/+185 |
|/ / / | |||||
| | * | Don't filter equal nodes in reorder assists | Lukas Wirth | 2021-04-21 | 2 | -10/+6 |
| | | |