aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* Merge #3666bors[bot]2020-03-3011-110/+205
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3666: Reload part of the server configuration without restarts r=matklad a=SomeoneToIgnore Partially addresses https://github.com/rust-analyzer/rust-analyzer/issues/2857 Closes #3751 Reloads all server configuration that's not related to VFS without restarts. The VFS-related parameters are not considered, since VFS is planned to be rewritten/replaced in the future and I have a suspicion that with the current code, swapping the VFS and the file watchers on the fly will cause big troubles. I have to store and process the config request id separately, since the `workspace/configuration` response returns `any[]` (https://microsoft.github.io/language-server-protocol/specifications/specification-current/#workspace_configuration), if there's a better way to handle those responses, let me know. Co-authored-by: Kirill Bulatov <[email protected]>
| * Fix the endless progress bar issueKirill Bulatov2020-03-301-0/+10
| |
| * Rebase fixesKirill Bulatov2020-03-301-1/+1
| |
| * Avoid failing on incorrect settings responseKirill Bulatov2020-03-301-5/+13
| |
| * Small style fixKirill Bulatov2020-03-301-14/+21
| |
| * Code review fixesKirill Bulatov2020-03-305-18/+18
| | | | | | | | Co-Authored-By: Veetaha <[email protected]>
| * Small style fixKirill Bulatov2020-03-301-1/+1
| | | | | | Co-Authored-By: Veetaha <[email protected]>
| * Remove an obsolete fixmeKirill Bulatov2020-03-301-1/+0
| |
| * Reload watcher configurationKirill Bulatov2020-03-301-14/+19
| |
| * Style fixesKirill Bulatov2020-03-304-20/+31
| |
| * Reload only the properties that do not affect vfsKirill Bulatov2020-03-304-60/+89
| |
| * Read new config on the server sideKirill Bulatov2020-03-301-9/+18
| |
| * Send the config from the clientKirill Bulatov2020-03-302-24/+28
| |
| * Process configuration response draftKirill Bulatov2020-03-302-23/+24
| |
| * Client side draftKirill Bulatov2020-03-303-2/+14
|/
* Merge #3771bors[bot]2020-03-302-6/+5
|\ | | | | | | | | | | | | | | | | | | | | 3771: Use IntoIter r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * Use IntoIterAleksey Kladov2020-03-302-6/+5
|/
* Merge #3761bors[bot]2020-03-302-35/+164
|\ | | | | | | | | | | | | | | 3761: Append new match arms rather than replacing all of them r=matklad a=mattyhall This means we now retain comments when filling in match arms. This fixes #3687. This is my first contribution so apologies if it needs a rethink! I think in particular the way I find the position to append to and remove_if_only_whitespace are a little hairy. Co-authored-by: Matthew Hall <[email protected]>
| * Tidy up insertion position logicMatthew Hall2020-03-291-20/+6
| |
| * Remove unneeded variablesMatthew Hall2020-03-291-4/+2
| |
| * Remove "TODO" in comment in testMatthew Hall2020-03-281-4/+4
| |
| * Append new match arms rather than replacing all of themMatthew Hall2020-03-282-35/+180
| | | | | | | | This means we now retain comments when filling in match arms.
* | Merge #3763bors[bot]2020-03-301-11/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | 3763: Simplify r=matklad a=Veetaha Co-authored-by: veetaha <[email protected]>
| * | Simplifyveetaha2020-03-281-11/+4
| |/
* | Merge #3770bors[bot]2020-03-303-17/+14
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3770: Pull options outwards r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | Pull options outwardsAleksey Kladov2020-03-303-17/+14
| | |
* | | Merge #3754bors[bot]2020-03-305-44/+23
|\| | | | | | | | | | | | | | | | | | | | | | | 3754: Use automatic thread joining for cargo-watch r=matklad a=matklad r? @kiljacken Co-authored-by: Aleksey Kladov <[email protected]>
| * | Fix race in the testsAleksey Kladov2020-03-303-2/+11
| | |
| * | Use automatic thread joining for cargo-watchAleksey Kladov2020-03-303-40/+10
| | |
| * | :arrow_up: jod-threadAleksey Kladov2020-03-301-2/+2
|/ /
* | Merge pull request #3769 from matklad/btAleksey Kladov2020-03-301-0/+1
|\ \ | | | | | | Enable backtrace on CI
| * | Enable backtrace on CIAleksey Kladov2020-03-301-0/+1
| |/
* | Merge #3764bors[bot]2020-03-302-26/+26
|\ \ | |/ |/| | | | | | | | | | | | | | | 3764: Move roots_to_scan to LoopState r=matklad a=edwin0cheng closes #3760 cc @lnicola Co-authored-by: Edwin Cheng <[email protected]>
| * Fix review commentsEdwin Cheng2020-03-292-13/+13
| |
| * Move roots_to_scan to LoopStateEdwin Cheng2020-03-282-22/+22
|/
* Merge #3756bors[bot]2020-03-281-1/+1
|\ | | | | | | | | | | | | | | 3756: Update docs to mention vscode installation path on Windows r=edwin0cheng a=edwin0cheng Co-authored-by: Edwin Cheng <[email protected]>
| * Update docs to mention on WindowsEdwin Cheng2020-03-281-1/+1
|/
* Merge #3755bors[bot]2020-03-281-1/+1
|\ | | | | | | | | | | | | | | | | 3755: Update docs to mention vscode installation path on macOS r=matklad a=klochowicz It took me a while to find it on macOS so I thought I'd spare the effort for others ;) Co-authored-by: Mariusz Klochowicz <[email protected]> Co-authored-by: Aleksey Kladov <[email protected]>
| * Update docs/user/readme.adocAleksey Kladov2020-03-281-1/+1
| | | | | | Co-Authored-By: Laurențiu Nicola <[email protected]>
| * Update docs to mention vscode installation path on macOSMariusz Klochowicz2020-03-281-1/+1
|/
* Merge #3753bors[bot]2020-03-2827-153/+229
|\ | | | | | | | | | | | | | | | | 3753: Introduce stdx crate r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * Remove some unwrapsAleksey Kladov2020-03-285-29/+29
| |
| * Nice string formattingAleksey Kladov2020-03-2811-21/+38
| |
| * Start stdxAleksey Kladov2020-03-2813-104/+163
|/ | | | This crate will hold everything to small to be worth publishing
* Merge #3745bors[bot]2020-03-273-2/+36
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3745: Fix merge-imports assist for wildcard imports r=matklad a=piotr-szpetkowski Refs #3728 Besides the case mentioned in issue merging two diff-prefix wildcard uses will now work as well e.g. ```rust use std::cell::*; use std::str::*; ``` will translate into: ```rust use std::{cell::*, str::*} ``` I'd also like to explore usage of the `merge-imports` for same-prefix uses to simplify redundancy, but it seems like an idea for another issue and I'm not sure if it's something that this assist should do e.g.: ```rust use std::cell::Cell; use std::cell::*; ``` into: ```rust use std::cell::*; ``` Co-authored-by: Piotr Szpetkowski <[email protected]>
| * Fix merge-imports assist for wildcard importsPiotr Szpetkowski2020-03-273-2/+36
|/
* Merge #3743bors[bot]2020-03-272-5/+9
|\ | | | | | | | | | | | | | | | | | | | | 3743: Fix assist description r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * Fix assist descriptionAleksey Kladov2020-03-272-5/+9
|/
* Merge #3742bors[bot]2020-03-276-5/+170
|\ | | | | | | | | | | | | | | | | | | | | 3742: Replace if with if-let r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * Replace if with if-letAleksey Kladov2020-03-276-5/+170
| |
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-09 23:38:42 +0000