aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
*-. 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]>
| * | | Fixed formattingBastian Kersting2021-04-261-1/+1
| | | |
| * | | Added mutable semantic token modifier for &mut self methodsBastian Kersting2021-04-262-4/+9
|/ / /
* | | Merge #8658bors[bot]2021-04-252-6/+105
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8658: Check more carefully for cases where a rename can't be done r=Veykril a=dzhu Attempting to rename an element of a tuple field would previously replace the type with the new name, which doesn't make sense; now it fails instead. The check is done in both `prepare_rename` and `rename` so that the case is caught before the user is prompted for a new name. Some other existing failure cases are also now additionally checked in `prepare_rename`. Closes: #8592 (I threw in some doc edits for a relevant type; of course, I can remove those if the policy here is to be strict about scope of changes within a PR.) Co-authored-by: Danny Zhu <[email protected]>
| * | | Check more carefully for cases where a rename can't be doneDanny Zhu2021-04-252-6/+105
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Attempting to rename an element of a tuple field would previously replace the type with the new name, which doesn't make sense; now it fails instead. The check is done in both `prepare_rename` and `rename` so that the case is caught before the user is prompted for a new name. Some other existing failure cases are also now additionally checked in `prepare_rename`.
* | | Merge #8656bors[bot]2021-04-251-4/+4
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 8656: Update deps r=kjeremy a=kjeremy Co-authored-by: Jeremy Kolb <[email protected]>
| * | Update depsJeremy Kolb2021-04-251-4/+4
|/ /
* | Merge #8647bors[bot]2021-04-248-329/+344
|\ \ | | | | | | | | | | | | | | | | | | | | | 8647: internal: Split out merge_imports module from helpers::insert_use r=Veykril a=Veykril bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | Split out merge_imports module from helpers::insert_useLukas Wirth2021-04-248-329/+344
|/ /
* | Merge #8643bors[bot]2021-04-242-24/+46
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 8643: fix: correct version string to contain hash, build date and channel r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | fix: correct version string to contain hash, build date and channelAleksey Kladov2021-04-242-24/+46
|/ /
* | minorAleksey Kladov2021-04-241-1/+1
| |
* | Merge #8641bors[bot]2021-04-241-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 8641: minor r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | minorAleksey Kladov2021-04-241-1/+1
| | |
* | | Merge #8639bors[bot]2021-04-231-1/+3
|\| | | | | | | | | | | | | | | | | | | | | | | | | | 8639: fix: 'configuration.property' error in VS Code r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | fix: 'configuration.property' error in VS CodeAleksey Kladov2021-04-231-1/+3
| | | | | | | | | | | | closes #7789
| | |
| \ \
*-. \ \ Merge #8591 #8638bors[bot]2021-04-2315-355/+331
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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_fnLukas Wirth2021-04-233-31/+24
| |/ / |/| |
* | | Merge #8637bors[bot]2021-04-237-150/+147
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8637: Tag `yield` and `await` as ControlFlow in semantic highlighting r=Veykril a=Veykril and shuffle some code around Closes #8634 bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | | Simplify highlight token match guardsLukas Wirth2021-04-232-102/+92
| | | |
| * | | Sort HlMod variants and ALL constLukas Wirth2021-04-236-46/+45
| | | |
| * | | Tag `yield` and `await` as ControlFlow in semantic highlightingLukas Wirth2021-04-232-5/+13
|/ / /
* | | Merge #8317bors[bot]2021-04-235-7/+636
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | 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" assistunexge2021-04-231-0/+40
| | |
| * | Stop iterating reference after made an edit in "Convert to named struct" assistunexge2021-04-231-2/+47
| | |