aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | Add **Ignore Test** assistAleksey Kladov2020-11-176-18/+80
|/ / / /
* | | | Merge #6571bors[bot]2020-11-173-108/+75
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6571: Cleanup project model r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | Remove needless allocAleksey Kladov2020-11-171-1/+1
| | | | |
| * | | | Compress codeAleksey Kladov2020-11-172-66/+42
| | | | |
| * | | | simplifyAleksey Kladov2020-11-171-41/+32
| | |/ / | |/| |
* | | | Merge #6566bors[bot]2020-11-1712-33/+32
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 6566: Latest LSP 3.16 protocol r=matklad a=kjeremy Pulls in https://github.com/gluon-lang/lsp-types/pull/186 Co-authored-by: kjeremy <[email protected]>
| * | | Latest LSP 3.16 protocolkjeremy2020-11-1612-33/+32
|/ / / | | | | | | | | | Pulls in https://github.com/gluon-lang/lsp-types/pull/186
* | | Merge #6565bors[bot]2020-11-161-1/+2
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | 6565: Prepare to recompute completions on every keystroke r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | Prepare to recompute completions on every keystrokeAleksey Kladov2020-11-161-1/+2
|/ / | | | | | | | | If we set `is_incomplete: true`, VS Code will re-query completions after every keypress.
* | Merge #6563bors[bot]2020-11-161-0/+46
|\ \ | | | | | | | | | | | | | | | | | | | | | 6563: Don't complete keywords in struct initializers r=matklad a=Veykril Fixes #6562 Co-authored-by: Lukas Wirth <[email protected]>
| * | Don't complete keywords in struct literalsLukas Wirth2020-11-161-0/+46
| | |
* | | Merge #6558bors[bot]2020-11-163-0/+26
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | 6558: format string highlighting: handle hex + debug type specifier r=matklad a=ruabmbua Should fix https://github.com/rust-analyzer/rust-analyzer/issues/6427 Co-authored-by: Roland Ruckerbauer <[email protected]>
| * | format string highlighting: handle hex + debug type specifierRoland Ruckerbauer2020-11-153-0/+26
| | |
* | | Merge #6552bors[bot]2020-11-155-26/+206
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 6552: Properly handle shorthands in destructure patterns when renaming r=SomeoneToIgnore a=Veykril Fixes #6548 and #6551. Co-authored-by: Lukas Wirth <[email protected]>
| * | Cleanup edit_text_range_for_record_field_expr_or_patLukas Wirth2020-11-151-46/+59
| | |
| * | Use shorthand field syntax in destructuresLukas Wirth2020-11-143-29/+84
| | |
| * | Properly handle shorthands in destructure patterns when renamingLukas Wirth2020-11-142-6/+69
| | |
| * | Use shorthand record syntax when renaming struct initializer fieldLukas Wirth2020-11-144-19/+68
| | |
* | | Merge #6544bors[bot]2020-11-134-4/+118
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6544: add suggestion ..Default::default() for remaining struct fields in a constructor r=bnjjj a=bnjjj I'm not sure I should import `assists` crate inside `completions`, maybe we should move out `FamousDefs` from `assists` ? Let me know :) close #6492 Signed-off-by: Benjamin Coenen <[email protected]> Co-authored-by: Benjamin Coenen <[email protected]>
| * | | add suggestion ..Default::default() for remaining struct fields in a ↵Benjamin Coenen2020-11-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | constructor #6492 Signed-off-by: Benjamin Coenen <[email protected]>
| * | | add suggestion ..Default::default() for remaining struct fields in a ↵Benjamin Coenen2020-11-134-4/+118
| | | | | | | | | | | | | | | | | | | | | | | | constructor #6492 Signed-off-by: Benjamin Coenen <[email protected]>
* | | | Merge #6545bors[bot]2020-11-138-630/+618
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6545: Simplify project model r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | Remove dead codeAleksey Kladov2020-11-131-15/+0
| | | | |
| * | | | Cleanup workspace loading a tiny bitAleksey Kladov2020-11-137-46/+40
| | | | |
| * | | | Move tricky workspace logic to a separate moduleAleksey Kladov2020-11-132-602/+611
| | | | |
* | | | | Merge #6543bors[bot]2020-11-131-0/+0
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6543: cleanup r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | cleanupAleksey Kladov2020-11-131-0/+0
|/ / / /
* | | | Merge #6519bors[bot]2020-11-1311-8/+105
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6519: Add "Open Cargo.toml" action r=matklad a=p3achyjr ## What is it? This adds an "open cargo.toml" action from the vscode shell, resolves #6462 ## Test Ran ```cargo xtask install --server``` and ```cargo xtask install --client```, then ```Developer: Reload Window```. ![image](https://user-images.githubusercontent.com/8763808/98614382-2a578f00-22ad-11eb-9811-4a72a54ff6fb.png) When clicked: ![image](https://user-images.githubusercontent.com/8763808/98618176-77d7fa00-22b5-11eb-8788-35256542f3a6.png) Co-authored-by: Anatol Liu <[email protected]>
| * | | | add open Cargo.toml actionAnatol Liu2020-11-1315-42/+114
| | | | |
* | | | | Merge #6542bors[bot]2020-11-132-2755/+11
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6542: Revert "Upgrade version of npms lockfile" r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | Revert "Upgrade version of npms lockfile"Aleksey Kladov2020-11-132-2755/+11
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 3d559afc11b230d52b13c5540706b85a7eeb3490. VS Code uses nodejs 12 still, so its better to stick to the format it understands.
* | | | Merge #6538bors[bot]2020-11-122-11/+2755
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6538: Upgrade version of npms lockfile r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | Upgrade version of npms lockfileAleksey Kladov2020-11-122-11/+2755
|/ / / /
* | | | Merge #6524bors[bot]2020-11-125-69/+233
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6524: Add support for loading rustc private crates r=matklad a=xldenis This PR presents a solution to the problem of making`rustc_private` crates visible to `rust-analyzer`. Currently developers add dependencies to those crates behind a `cfg(NOT_A_TARGET)` target to prevent `cargo` from building them. This solution is unsatisfactory since it requires modifying `Cargo.toml` and causes problems for collaboration or CI. The proposed solution suggested by @matklad is to allow users to give RA a path where the `rustc` sources could be found and then load that like a normal workspace. This PR implements this solution by adding a `rustcSource` configuration item and adding the cargo metadata to the crate graph if it is provided. ------ I have no idea how this should be tested, or if this code is generally tested at all. I've locally run the extension with these changes and it correctly loads the relevant crates on a `rustc_private` project of mine. Co-authored-by: Xavier Denis <[email protected]>
| * | | | Address review commentsXavier Denis2020-11-123-3381/+17
| | | | |
| * | | | Reorder rustc_private loadingXavier Denis2020-11-111-87/+86
| | | | |
| * | | | Add support for loading rustc private cratesXavier Denis2020-11-116-79/+3608
| | | | |
* | | | | Merge #6514bors[bot]2020-11-121-28/+92
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6514: Fix extract_struct_from_enum_variant not updating record references r=Veykril a=Veykril Related to #6510 Co-authored-by: Lukas Wirth <[email protected]>
| * | | | | Add extra test to extract_struct_from_enum_variantLukas Wirth2020-11-121-1/+28
| | | | | |
| * | | | | Use Module::find_use_path_prefixed in extract_struct_from_enum_variant assistLukas Wirth2020-11-121-5/+9
| | | | | |
| * | | | | Fix extract_struct_from_enum_variant not updating record referencesLukas Wirth2020-11-121-26/+59
| | | | | |
* | | | | | Merge #6513bors[bot]2020-11-121-1/+82
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6513: Support qualified function calls in remove_unused_param r=Veykril a=Veykril Also adds a test to check that it removes unused params across files. Co-authored-by: Lukas Wirth <[email protected]>
| * | | | | | Support qualified function calls in remove_unused_paramLukas Wirth2020-11-121-1/+49
| | | | | | |
| * | | | | | Add multiple file edit tes to remove_unused_param assistLukas Wirth2020-11-091-0/+33
| |/ / / / /
* | | | | | Merge #6529bors[bot]2020-11-121-19/+77
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6529: Do not insert imports before inner comments r=Veykril a=SomeoneToIgnore Co-authored-by: Kirill Bulatov <[email protected]>
| * | | | | Remove the fixmeKirill Bulatov2020-11-121-36/+15
| | | | | |
| * | | | | Add a FIXME for non-unified inner attributesKirill Bulatov2020-11-121-2/+7
| | | | | |
| * | | | | Do not insert imports before inner commentsKirill Bulatov2020-11-121-18/+92
|/ / / / /
* | | | | Merge #6534bors[bot]2020-11-124-47/+56
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6534: Fix attachment of inner doc comments r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | Fix attachment of inner doc commentsAleksey Kladov2020-11-123-19/+33
| | | | | |