aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | Merge #6999bors[bot]2020-12-220-0/+0
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6999: Build aarch64-apple-darwin binaries on CI (take 2) r=matklad a=lnicola Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | Build aarch64-apple-darwin binaries on CILaurențiu Nicola2020-12-222-32/+154
| | |/ / | |/| |
* | | | Merge #6989bors[bot]2020-12-222-32/+154
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 6989: Build aarch64-apple-darwin binaries on CI r=matklad a=lnicola This splits the `dist` matrix job into four and tries to make `xtask dist` more principled about target and artifact naming. Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | Build aarch64-apple-darwin binaries on CILaurențiu Nicola2020-12-222-32/+154
| | | |
* | | | Merge #6746bors[bot]2020-12-2210-19/+252
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6746: Feature/add assist extract module to file r=matklad a=sasurau4 Fix #6522 ## Screenshot <img src="https://user-images.githubusercontent.com/13580199/102748269-33a44300-43a5-11eb-9e37-f5fcb8e62f73.gif" width=600 /> ## TODO - [x] Remove all TODO comment - [x] Pass the doc test Co-authored-by: Daiki Ihara <[email protected]>
| * | | | Use workspace applyEdit instead of snippetWorkspaceEditDaiki Ihara2020-12-211-1/+1
| | | | |
| * | | | Fix assist test logicDaiki Ihara2020-12-211-2/+6
| | | | |
| * | | | Update test diagnostics in ideDaiki Ihara2020-12-211-0/+1
| | | | |
| * | | | Add initial_contents field for CreateFileDaiki Ihara2020-12-215-50/+47
| | | | |
| * | | | Add extract_module_to_file assistDaiki Ihara2020-12-213-0/+187
| | | | |
| * | | | Add handling file_system_edit in test assistsDaiki Ihara2020-12-212-15/+59
| | | | |
* | | | | Merge #6984bors[bot]2020-12-223-1146/+0
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6984: Remove TextMate grammar r=dustypomerleau a=lnicola Closes #6267 This is now included upstream in VS Code. Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | Remove TextMate grammarLaurențiu Nicola2020-12-213-1146/+0
| | | | | | | | | | | | | | | | | | | | It's now included upstream in VS Code
* | | | | Merge #6991bors[bot]2020-12-211-1/+3
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6991: Don't download x64 binaries on non-Mac arm64 r=lnicola a=lnicola bors r+ Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | | Don't download x64 binaries on non-Mac arm64Laurențiu Nicola2020-12-211-1/+3
|/ / / / /
* | | | | Merge #6990bors[bot]2020-12-211-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6990: Allow code extension to download binary in apple arm r=lnicola a=phungleson Apple chip has a new `arch`. It subsequently downloads Intel version of `rust-analyzer` but it works regardless. Co-authored-by: Son <[email protected]>
| * | | | | Allow code extension to download binary in apple armSon2020-12-211-1/+1
|/ / / / /
* | | | | Merge #6988bors[bot]2020-12-211-3/+2
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6988: Remove outdated FIXME r=lnicola a=lnicola bors r+ Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | Remove outdated FIXMELaurențiu Nicola2020-12-211-3/+2
|/ / / /
* | | | Merge #6985bors[bot]2020-12-211-1/+10
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6985: Use /etc/os-release to check for NixOS r=matklad a=lnicola Closes #5641 The motivation in #5641 isn't too strong, but /etc/os-release exists on pretty much every Linux distro, while /etc/nixos sounds like an implementation detail. Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | Use /etc/os-release to check for NixOSLaurențiu Nicola2020-12-211-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The motivation in #5641 isn't too strong, but /etc/os-release exists on pretty much every Linux distro, while /etc/nixos sounds like an implementation detail.
* | | | | Merge #6987bors[bot]2020-12-211-7/+74
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6987: Delete related whitespace when removing unused param r=matklad a=rherrmann Include adjacent whitespace in the text edits to remove the parameter declaration and its occurences in calling code. https://github.com/rust-analyzer/rust-analyzer/issues/6663 Co-authored-by: Rüdiger Herrmann <[email protected]>
| * | | | Delete related whitespace when removing unused paramRüdiger Herrmann2020-12-211-7/+74
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | Include adjacent whitespace in the text edits to remove the parameter declaration and its occurences in calling code. https://github.com/rust-analyzer/rust-analyzer/issues/6663
* | | | Merge #6982bors[bot]2020-12-212-1/+16
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6982: Remove parentheses when inverting `!(cond)` r=matklad a=Jesse-Bakker Followup to #6894 When inverting a composite condition twice, the parentheses were left. This also removes those unnecessary parentheses when applying the invert-if assist. Co-authored-by: Jesse Bakker <[email protected]>
| * | | Remove parentheses when inverting `!(cond)`Jesse Bakker2020-12-212-1/+16
| | | |
* | | | Merge #6981bors[bot]2020-12-211-6/+42
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6981: Acquire original text range when searching for references r=Veykril a=Veykril Fixes #6976 Co-authored-by: Lukas Wirth <[email protected]>
| * | | | Acquire original text range when searching for referencesLukas Wirth2020-12-211-6/+42
|/ / / /
* | | | Merge #6978bors[bot]2020-12-211-0/+4
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6978: Filter out empty rustc spans r=lnicola a=jonas-schievink Hopefully, this fixes https://github.com/rust-analyzer/rust-analyzer/issues/6892 (I couldn't test it since r-a currently requires a too recent version of VS Code) Co-authored-by: Jonas Schievink <[email protected]>
| * | | | Filter out empty rustc spansJonas Schievink2020-12-211-0/+4
| |/ / /
* | | | Merge #6977bors[bot]2020-12-211-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6977: Revert "Stop setting CompletionItem::deprecated" r=lnicola a=lnicola We should keep setting it, according to https://github.com/rust-analyzer/rust-analyzer/pull/6974#issuecomment-748983789. Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | Revert "Stop setting CompletionItem::deprecated"Laurențiu Nicola2020-12-211-1/+1
|/ / / / | | | | | | | | | | | | This reverts commit 299ce609952dfaa6bef86e967c5632e8e42b8fca.
* | | | Merge #6974bors[bot]2020-12-211-1/+1
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6974: Stop setting CompletionItem::deprecated r=matklad a=lnicola Closes #2042 We're now using the `CompletionItem::tags` field to mark `CompletionItem`s as deprecated, and `CompletionItem::deprecated` is gone from LSP. Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | Stop setting CompletionItem::deprecatedLaurențiu Nicola2020-12-211-1/+1
|/ / /
* | | Merge #6965bors[bot]2020-12-217-147/+199
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6965: Properly attach attributes to Param instead of parent ParamList r=matklad a=Veykril Fixes #2783, fixes #2781 The problem with `let _a = [0,#[cfg(feature = "L")]0];` has already been fixed some time ago it seems: <details> <summary>Syntax Tree for the const item</summary> ``` [email protected] [email protected] "let" [email protected] " " [email protected] [email protected] [email protected] "_a" [email protected] " " [email protected] "=" [email protected] " " [email protected] [email protected] "[" [email protected] [email protected] "0" [email protected] "," [email protected] [email protected] [email protected] "#" [email protected] "[" [email protected] [email protected] [email protected] [email protected] "cfg" [email protected] [email protected] "(" [email protected] "feature" [email protected] " " [email protected] "=" [email protected] " " [email protected] "\"L\"" [email protected] ")" [email protected] "]" [email protected] "0" [email protected] "]" [email protected] ";" ``` </details> Co-authored-by: Lukas Wirth <[email protected]>
| * | | Parse attributes in tuple expressionsLukas Wirth2020-12-203-3/+56
| | | |
| * | | Properly attach attributes to Param instead of parent ParamListLukas Wirth2020-12-204-144/+143
| | |/ | |/|
* | | Merge #6967bors[bot]2020-12-214-9/+63
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | 6967: Correctly parse legacy trait objects with leading ForType r=matklad a=Veykril Fixes #1422 Co-authored-by: Lukas Wirth <[email protected]>
| * | Properly parse legacy trait objects with leading ForTypeLukas Wirth2020-12-204-9/+63
| |/
* | Merge #6968bors[bot]2020-12-201-4/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | 6968: Update syn and quote r=kjeremy a=kjeremy Co-authored-by: Jeremy Kolb <[email protected]>
| * | Update syn and quoteJeremy Kolb2020-12-201-4/+4
| | |
* | | Merge #6963bors[bot]2020-12-201-15/+27
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6963: Split lines between sentences r=lnicola a=lnicola bors r+ Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | Split lines between sentencesLaurențiu Nicola2020-12-201-15/+27
| | | |
* | | | Merge #6961bors[bot]2020-12-201-1/+1
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6961: Document the version requirements for building from source r=lnicola a=lnicola Fixes #6667 bors r+ Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | Document the version requirements for building from sourceLaurențiu Nicola2020-12-201-1/+1
| | | |
* | | | Merge #6959bors[bot]2020-12-203-2/+14
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6959: Use pattern_single instead of pattern in mbe pat r=jonas-schievink a=edwin0cheng Fix #6882 Co-authored-by: Edwin Cheng <[email protected]>
| * | | | Use pattern_single instead of pattern in mbe patEdwin Cheng2020-12-203-2/+14
| | |_|/ | |/| |
* | | | Merge #6921bors[bot]2020-12-205-19/+74
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 6921: Higher-ranked trait bounds for where clauses r=flodiebold a=Veykril There is a slight problem with this which is also noted in a FIXME now but `LifetimeParameters` of these ForLifetime where clauses allocate the lifetimes in the corresponding arena as if they were lifetimes of the item itself and not just the clause they belong to. I wasn't entirely sure what I could do about this but given nothing really uses lifetimes like that currently I figured it might be fine? Open to suggestions for that problem. Co-authored-by: Lukas Wirth <[email protected]>
| * | | Higher-ranked trait bounds for where clausesLukas Wirth2020-12-174-18/+74
| | | |
| * | | Remove obsolete FIXMELukas Wirth2020-12-171-1/+0
| | | |
* | | | Merge #6958bors[bot]2020-12-2022-102/+88
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6958: Align code_model name with ungrammar r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>