Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | minor: use minicore | Aleksey Kladov | 2021-06-18 | 1 | -11/+1 |
| | |||||
* | internal: add Copy to minicore | Aleksey Kladov | 2021-06-18 | 1 | -302/+409 |
| | |||||
* | Merge #9321 | bors[bot] | 2021-06-18 | 3 | -165/+2 |
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9321: Inline generics in const and function trait completions r=Veykril a=RDambrosio016 This PR does a couple of things: - moves path_transform from ide_assists to ide_db to be shared by both assists and completions - when completing a const or a function for a trait, it will "inline" any generics in those associated items instead of leaving the generic's name. For example: ```rust trait Foo<T> { const BAR: T; fn foo() -> T; } struct Bar; impl Foo<u32> for Bar { // autocompletes to this fn foo() -> u32; // and not this (old) fn foo() -> T; // also works for associated consts and where clauses const BAR: u32 = /* */ } ``` Currently this does not work for const generics, because `PathTransform` does not seem to account for them. If this should work on const generics too, `PathTransform` will need to be changed. However, it is uncommon to implement a trait only for a single const value, so this isnt a huge concern. Co-authored-by: rdambrosio <[email protected]> | ||||
| * | Feat: inline generics in const and func trait completions | rdambrosio | 2021-06-18 | 3 | -165/+2 |
| | | |||||
* | | Minor clippy perf fixes | Laurențiu Nicola | 2021-06-18 | 1 | -2/+2 |
|/ | |||||
* | minor: simplify | Aleksey Kladov | 2021-06-17 | 1 | -15/+12 |
| | |||||
* | internal: add derive and ord support to minicore | Aleksey Kladov | 2021-06-17 | 2 | -48/+67 |
| | |||||
* | internal: add From to minicore | Aleksey Kladov | 2021-06-17 | 2 | -50/+99 |
| | |||||
* | internal: add default to minicore | Aleksey Kladov | 2021-06-17 | 2 | -49/+75 |
| | |||||
* | simplify | Lukas Wirth | 2021-06-17 | 1 | -9/+7 |
| | |||||
* | Nest all the or-patterns! | Lukas Wirth | 2021-06-17 | 3 | -26/+32 |
| | |||||
* | Merge #9308 | bors[bot] | 2021-06-17 | 1 | -13/+52 |
|\ | | | | | | | | | | | | | | | 9308: fix: Create modules in correct directory for nested modules in move_module assist r=Veykril a=Veykril bors r+ Co-authored-by: Lukas Wirth <[email protected]> | ||||
| * | Create modules in correct directory for nested modules in move_module assist | Lukas Wirth | 2021-06-17 | 1 | -13/+52 |
| | | |||||
* | | internal: switch some tests to minicore | Aleksey Kladov | 2021-06-17 | 1 | -84/+37 |
| | | |||||
* | | internal: minimize minicore | Aleksey Kladov | 2021-06-17 | 1 | -53/+25 |
| | | | | | | | | | | | | We want to keep minicore small, so let's split out iterator adapters and sources into a separate `iterators` region, and use them only when needed. | ||||
* | | internal: cleanup tests | Aleksey Kladov | 2021-06-16 | 1 | -71/+114 |
| | | |||||
* | | internal: switch some tests to minicore | Aleksey Kladov | 2021-06-16 | 1 | -15/+12 |
| | | |||||
* | | internal: switch some tests to minicore | Aleksey Kladov | 2021-06-16 | 1 | -19/+15 |
|/ | |||||
* | internal: don't #[ignore] tests | Aleksey Kladov | 2021-06-15 | 2 | -35/+16 |
| | | | | See the style.md for motivation | ||||
* | internal: cleanup tests | Aleksey Kladov | 2021-06-15 | 2 | -610/+596 |
| | | | | | | * ensure standard, non-indented style (should add this check to `fixture` some day) * removed a couple of ignores | ||||
* | internal: document that we don't #[ignore] tests | Aleksey Kladov | 2021-06-15 | 1 | -16/+16 |
| | |||||
* | internal: cut deps between assists and diagnostics | Aleksey Kladov | 2021-06-14 | 1 | -127/+4 |
| | |||||
* | internal: prepare to move assist definitions | Aleksey Kladov | 2021-06-14 | 2 | -29/+27 |
| | |||||
* | Don't keep a trailing self token in import paths after unmerge_use | Lukas Wirth | 2021-06-14 | 1 | -1/+15 |
| | |||||
* | Merge #9242 | bors[bot] | 2021-06-13 | 22 | -37/+36 |
|\ | | | | | | | | | | | | | | | 9242: Clippy r=matklad a=Maan2003 Best viewed commit wise Co-authored-by: Maan2003 <[email protected]> | ||||
| * | cargo fmt | Maan2003 | 2021-06-13 | 1 | -2/+1 |
| | | |||||
| * | clippy::redundant_field_names | Maan2003 | 2021-06-13 | 1 | -1/+1 |
| | | |||||
| * | clippy::useless_return | Maan2003 | 2021-06-13 | 1 | -4/+4 |
| | | |||||
| * | clippy::redundant_closure | Maan2003 | 2021-06-13 | 2 | -3/+3 |
| | | |||||
| * | clippy::redudant_borrow | Maan2003 | 2021-06-13 | 19 | -28/+28 |
| | | |||||
* | | Merge #9240 | bors[bot] | 2021-06-13 | 1 | -1/+1 |
|\ \ | |/ |/| | | | | | | | | | | | 9240: Fixed a small typo in comment r=matklad a=fee1-dead Co-authored-by: fee1-dead <[email protected]> | ||||
| * | Fixed a small typo in comment | fee1-dead | 2021-06-11 | 1 | -1/+1 |
| | | |||||
* | | Merge #9215 | bors[bot] | 2021-06-11 | 1 | -2/+9 |
|\ \ | |/ |/| | | | | | | | | | | | 9215: change visibility for use and macro items r=jonas-schievink a=Maan2003 Co-authored-by: Maan2003 <[email protected]> | ||||
| * | change visibility for use and macro items | Maan2003 | 2021-06-11 | 1 | -2/+9 |
| | | |||||
* | | Don't suggest Remove unused param in trait impls | Maan2003 | 2021-06-11 | 1 | -1/+29 |
|/ | |||||
* | fmt | Joshua Warner | 2021-06-06 | 1 | -3/+12 |
| | |||||
* | Refactor to be just one assist | Joshua Warner | 2021-06-05 | 3 | -532/+328 |
| | |||||
* | Add assist for converting a tuple enum variant to a named variant | Joshua Warner | 2021-06-05 | 2 | -0/+517 |
| | |||||
* | Don't inline mutable locals in 'inline_local_variable' | Lukas Wirth | 2021-06-04 | 1 | -0/+17 |
| | |||||
* | NFC: remove redundant clones (clippy::perf) | Matthias Krüger | 2021-06-03 | 1 | -1/+1 |
| | |||||
* | Preserve indentation | Jonas Schievink | 2021-06-02 | 1 | -5/+30 |
| | |||||
* | Account for traits | Jonas Schievink | 2021-06-02 | 1 | -10/+29 |
| | |||||
* | Merge #9108 | bors[bot] | 2021-06-02 | 1 | -2/+31 |
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9108: Don't show extract into variable assist for unit expressions r=jonas-schievink a=brandondong **Reproduction:** ```rust fn main() { let mut i = 3; $0if i >= 0 { i += 1; } else { i -= 1; }$0 } ``` 1. Select the snippet of code between the $0's. 2. The extract into variable assist shows up, pushing down the more useful extract into function assist. 3. The resulting output of selecting the extract into variable assist is valid but with the extracted variable having the unit type: ```rust fn main() { let mut i = 3; let var_name = if i >= 0 { i += 1; } else { i -= 1; }; var_name } ``` **Fix:** - Don't show the extract into variable assist for unit expressions. I could not think of any scenarios where such a variable extraction would be desired. Co-authored-by: Brandon <[email protected]> | ||||
| * | Don't show extract into variable assist for unit expressions | Brandon | 2021-06-02 | 1 | -2/+31 |
| | | |||||
* | | Merge #9112 | bors[bot] | 2021-06-02 | 1 | -44/+125 |
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9112: Fix some bugs in `extract_struct_from_enum_variant` assist r=Veykril a=Veykril bors r+ Fixes #9100 Fixes #9099 Kind of fixes #9109, it now copies all the generics might be incorrect if the variant doesn't use all of them) Co-authored-by: Lukas Wirth <[email protected]> | ||||
| * | | Fix references to patterns in extract_struct_from_enum_variant assist | Lukas Wirth | 2021-06-02 | 1 | -26/+51 |
| | | | |||||
| * | | Carry over attributes in extract_struct_from_enum_variant | Lukas Wirth | 2021-06-02 | 1 | -5/+32 |
| | | | |||||
| * | | Account for generics in extract_struct_from_enum_variant | Lukas Wirth | 2021-06-02 | 1 | -17/+46 |
| |/ | |||||
* / | Place extracted type alias outside of impl | Jonas Schievink | 2021-06-02 | 1 | -1/+27 |
|/ | |||||
* | clippy::redundant_clone fixes | Laurențiu Nicola | 2021-05-26 | 1 | -1/+1 |
| |