aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge #8701bors[bot]2021-05-011-1/+1
|\ | | | | | | | | | | | | | | | | 8701: Fix feature description inconsistency r=Veykril a=brandondong ![image](https://user-images.githubusercontent.com/13722457/116768676-4d0d6980-a9ed-11eb-8d7e-c7d340a99064.png) Just something that bugged me :) Co-authored-by: Brandon <[email protected]>
| * Fix feature description inconsistencyBrandon2021-05-011-1/+1
|/
* Merge #8698bors[bot]2021-04-301-2/+28
|\ | | | | | | | | | | | | | | | | | | 8698: fix: search only crate-level prelude when looking for an item r=jonas-schievink a=jonas-schievink Fixes https://github.com/rust-analyzer/rust-analyzer/issues/8688 bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * find_path: check only crate-level preludeJonas Schievink2021-04-301-2/+28
|/
* Merge #8692bors[bot]2021-04-304-25/+100
|\ | | | | | | | | | | | | | | | | | | 8692: Fix panic caused by new Try trait definition r=flodiebold a=flodiebold The new Try trait definition caused a query cycle for us. This adds recovery for that cycle, but also fixes the cause, which is that we went through the supertraits when resolving `<T as Trait>::Assoc`, which isn't actually necessary. I also rewrote `all_super_trait_refs` to an iterator before I realized what the actual problem was, so I kept that. Fixes #8686. Co-authored-by: Florian Diebold <[email protected]>
| * Don't look in super traits for <T as Trait>::AssocFlorian Diebold2021-04-292-7/+14
| | | | | | | | | | | | | | This isn't actually how it works, you have to specify the exact trait that has the associated type. Fixes #8686.
| * Rewrite all_super_trait_refs as an iteratorFlorian Diebold2021-04-291-18/+34
| | | | | | | | | | Doesn't fix the bug I was trying to fix, but now that I did it anyway it seems fine to keep.
| * Handle cycles in generic_defaults more gracefullyFlorian Diebold2021-04-292-0/+22
| |
| * Add test for #8686Florian Diebold2021-04-291-0/+30
| |
* | Merge #8695bors[bot]2021-04-308-12/+12
|\ \ | |/ |/| | | | | | | | | | | | | 8695: internal: fix naming polarity r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * internal: fix naming polarityAleksey Kladov2021-04-304-6/+6
| | | | | | | | | | Type Constructors have *parameters*, when they are substituted with type *arguments*, we have a type.
| * internal: normalize nameAleksey Kladov2021-04-305-6/+6
| | | | | | | | All def types in hir are unsubstituted
* | Merge #8687bors[bot]2021-04-292-0/+40
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 8687: fix: closure unify without check ClosureId r=lnicola a=komonad Previously, the unification of closure types is blocked by `Ty.equals_ctor` which compares the ClosureId of the closures. Here is a workaround to allow closures to unify their substitutions. Fixes #8604. Co-authored-by: Comonad <[email protected]>
| * | fix: closure unify without check ClosureIdComonad2021-04-292-0/+40
|/ / | | | | | | closes #8604
* | Merge #8685bors[bot]2021-04-294-0/+141
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 8685: feat: parse const param defaults r=jonas-schievink a=jonas-schievink These will probably be stabilized in a few releases, let's make sure we can at least parse them bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | Parse const param defaultsJonas Schievink2021-04-294-0/+141
|/ /
* | Merge #8684bors[bot]2021-04-293-0/+40
|\ \ | |/ |/| | | | | | | | | | | | | | | 8684: Correctly parse negated literals as const args r=jonas-schievink a=jonas-schievink Previously we didn't accept `S::<-0>;` bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * Correctly parse negated literals as const argsJonas Schievink2021-04-293-0/+40
|/
* Merge #8683bors[bot]2021-04-283-13/+11
|\ | | | | | | | | | | | | | | | | 8683: internal: ensure that client and server-side configs are not mixed up r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * internal: ensure that client and server-side configs are not mixed upAleksey Kladov2021-04-283-13/+11
|/ | | | https://github.com/rust-lang/rust/issues/84647 would help big time here.
* Merge #8624bors[bot]2021-04-275-8/+60
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8624: Automatically detect rust library source file map r=vsrs a=vsrs This PR adds a new possible `rust-analyzer.debug.sourceFileMap` value: ```json { "rust-analyzer.debug.sourceFileMap": "auto" } ``` I did not make it the default because it uses two shell calls (`rustc --print sysroot` and `rustc -V -v`). First one can be slow (https://github.com/rust-lang/rustup/issues/783) Fixes #8619 Co-authored-by: vsrs <[email protected]>
| * Use explicit rustc commit-hashvsrs2021-04-223-3/+15
| | | | | | | | Required for lldb on mac
| * Add special `auto` value for `debug.sourceFileMap`vsrs2021-04-225-27/+38
| |
| * Autodetect rust library source file mapvsrs2021-04-222-2/+31
| |
| |
| \
*-. \ Merge #8629 #8657bors[bot]2021-04-272-404/+486
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8629: Allow pressing | to surround closure arguments in VSCode r=matklad a=Smittyvb This makes pressing <kbd>|</kbd> while some text is selected surround that text with pipes. This is useful when writing closures. In this demo, I press <kbd>|</kbd> to easily surround some text (`predicate`) with pipe characters to create a closure. Before, this would have replaced `predicate` with just `|`. ![](https://user-images.githubusercontent.com/10530973/115778847-3c257e00-a385-11eb-8c80-e0db970747db.gif) (I am getting failures when I do `cargo test` even on `master` locally (due to #8585 I think), but GH Actions is fine with this) 8657: npm update r=matklad a=kjeremy Co-authored-by: Smittyvb <[email protected]> Co-authored-by: Jeremy Kolb <[email protected]>
| | * | npm updateJeremy Kolb2021-04-251-407/+484
| | | |
| * | | Support | as a surrounding pairsSmittyvb2021-04-221-1/+2
| | | |
* | | | Merge #8679bors[bot]2021-04-271-0/+6
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8679: fix: don't show error message for a valid notification r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | fix: don't show error message for a valid notificationAleksey Kladov2021-04-271-0/+6
| | | | | | | | | | | | | | | | | | | | Closes #6782, #6772
* | | | | Merge #8677bors[bot]2021-04-272-3/+3
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8677: Return ServerNotInitialized according to the spec r=kjeremy a=kjeremy According to the spec we should return ServerNotInitialized if the server is waiting for an initialize request and something else comes in. Upgrading to lsp-server 0.5.1 will do this and retry until the initialize request comes in. Fixes #8581 Co-authored-by: Jeremy Kolb <[email protected]>
| * | | | | Return ServerNotInitialized according to the specJeremy Kolb2021-04-272-3/+3
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | According to the spec we should return ServerNotInitialized if the server is waiting for an initialize request and something else comes in. Upgrading to lsp-server 0.5.1 will do this and retry until the initialize request comes in. Fixes #8581
* | | | | Merge #8617bors[bot]2021-04-276-10/+59
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8617: Add option to opt out of smaller font size for inlay hints. r=SomeoneToIgnore a=jmederosalvarado As requested on issue #6883 this PR provides an option for users to opt out of the smaller font size for inlay hints. Part of #6883. Co-authored-by: Jorge Mederos Alvarado <[email protected]>
| * | | | | Fix how and when old inlay hint decorations are disposedJorge Mederos Alvarado2021-04-271-16/+33
| | | | | |
| * | | | | Add config optionsJorge Mederos Alvarado2021-04-272-0/+7
| | | | | |
| * | | | | Add option to opt out from smaller inlay hints font sizeJorge Mederos Alvarado2021-04-272-6/+30
| | | | | |
| * | | | | add option to package.jsonJorge Mederos Alvarado2021-04-272-4/+5
|/ / / / /
* | | | | Merge #8671bors[bot]2021-04-261-12/+3
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8671: minor: simplify r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | minor: simplifyAleksey Kladov2021-04-261-12/+3
| | | | | |
* | | | | | Merge #8670bors[bot]2021-04-261-1/+1
|\| | | | | | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8670: fix: make error message in the status bar more useful r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | fix: make error message in the status bar more usefulAleksey Kladov2021-04-261-1/+1
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | At the moment,the popup is just a bazillion of Cargo's "Compiling this\nCompiling that", which is not that useful. --quiet still displays error, which is what we needc
* | | | Merge #8668bors[bot]2021-04-263-3/+3
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8668: Use more cross-platform utc `date` argument r=matklad a=SomeoneToIgnore Part of https://github.com/rust-analyzer/rust-analyzer/issues/8571 ``` $ docker run -it --rm ubuntu:20.04 bash root@7393d1e7bbad:/# date -u +%Y-%m-%d 2021-04-26 ``` ``` $ date -u +%Y-%m-%d 2021-04-26 $ uname -a Darwin alaptop.local 20.3.0 Darwin Kernel Version 20.3.0: Thu Jan 21 00:07:06 PST 2021; root:xnu-7195.81.3~1/RELEASE_X86_64 x86_64 ``` Some of the places where I've change this do not really require it (since macos bin would have failed with `--iso` param also), but I've changed them for consistency. Co-authored-by: Kirill Bulatov <[email protected]>
| * | | | Use -u instead of --utc date flagKirill Bulatov2021-04-263-3/+3
|/ / / /
* | | | Merge #8666bors[bot]2021-04-265-30/+20
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8666: fix: correct version string on windows and mac r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | fix: add date to the version string on MacAleksey Kladov2021-04-262-2/+2
| | | | | | | | | | | | | | | | | | | | It doesn't support `--iso`
| * | | | fix: dont' misundentify nightly as stable in --version on Mac&WinAleksey Kladov2021-04-264-28/+18
|/ / / / | | | | | | | | | | | | | | | | We used to set `--nightly` in CI, and only for linux. Let's detect this in xtask instead.
* | | | Merge #8661bors[bot]2021-04-261-1/+10
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8661: internal: make normal release after a poin release less annoying r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | Update xtask/src/release.rsAleksey Kladov2021-04-261-1/+1
| | | | | | | | | | | | | | | Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | Update xtask/src/release.rsAleksey Kladov2021-04-261-1/+1
| | | | | | | | | | | | | | | Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | internal: make normal release after a poin release less annoyingAleksey Kladov2021-04-261-1/+10
|/ / / /
* | | | Merge #8660bors[bot]2021-04-262-4/+9
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8660: Added `mutable` semantic token modifier for `&mut self` methods r=Veykril a=1c3t3a This closes #8644. Co-authored-by: Bastian Kersting <[email protected]>