aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | LSP: Support EnumMember and Fieldkjeremy2019-03-111-0/+2
| | |/ | |/|
* | | Merge #962bors[bot]2019-03-122-6/+0
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 962: remove Def r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | remove DefAleksey Kladov2019-03-122-6/+0
|/ /
* | Merge #956bors[bot]2019-03-1010-2/+72
|\ \ | | | | | | | | | | | | | | | | | | | | | 956: Add async keyword r=c410-f3r a=c410-f3r Fixes #954. Co-authored-by: Caio <[email protected]>
| * | Add test for async blockCaio2019-03-103-1/+34
| | |
| * | Add async keywordCaio2019-03-098-2/+39
| | |
* | | Merge #957bors[bot]2019-03-102-23/+36
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | 957: Fix and simplify "rustc-watch" problem matcher r=vemoo a=vemoo Cargo watch now always ends with "[Finished running ..." and includes some extra info. Make the pattern only match "[Finished running". Also add "cargo watch" and "cargo watch tests" that use the "rustc-watch" problem matcher. Change the "Build Lsp" to do "cargo check" because that's faster, and I think enought in this case, and make it use "rustc" problem matcher from the extension. Co-authored-by: Bernardo <[email protected]>
| * | prettier formatBernardo2019-03-101-1/+1
| | |
| * | revert change to "check" since "build" is intentionalBernardo2019-03-101-1/+1
| | |
| * | add cargo watch tasksBernardo2019-03-101-22/+35
| | | | | | | | | | | | use extension problemMatcher
| * | simplify watch patternsBernardo2019-03-101-3/+3
|/ /
* | Merge #953bors[bot]2019-03-091-39/+61
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 953: Don't default publishDecorations and showWorkspaceLoaded to true on the server r=matklad a=flodiebold If the client doesn't specify these explicitly, that very likely means it doesn't know about them and so we shouldn't send them. In particular, the recent change to this default caused decorations to be sent to emacs, resulting in a lot of warning spam. Co-authored-by: Florian Diebold <[email protected]>
| * | Don't default publishDecorations to true on the serverFlorian Diebold2019-03-091-39/+61
|/ / | | | | | | | | | | | | If the client doesn't specify this explicitly, that very likely means it doesn't know about it and so we shouldn't send decorations. In particular, the recent change to this default caused decorations to be sent to emacs, resulting in a lot of warning spam.
* | Merge #955bors[bot]2019-03-081-5/+5
|\ \ | |/ |/| | | | | | | | | | | | | 955: Remove superfluous semicolons r=kjeremy a=kjeremy Doesn't change the result of the test but it does prevent the syntax tree from rendering. Co-authored-by: kjeremy <[email protected]>
| * Remove superfluous semicolonskjeremy2019-03-081-5/+5
|/ | | | | Doesn't change the result of the test but it does prevent the syntax tree from rendering.
* Merge #952bors[bot]2019-03-071-13/+13
|\ | | | | | | | | | | | | | | 952: Update some deps r=matklad a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * Update some depskjeremy2019-03-071-13/+13
|/
* Merge #951bors[bot]2019-03-071-3/+57
|\ | | | | | | | | | | | | | | | | | | | | | | | | 951: Fix EnumVariants not showing properly when hovering r=matklad a=vipentti This fixes #950 This also fixes hovering over the name of an EnumVariant. Additionally this changes the way enum variants are shown, now instead of the `Some<T>... -> Option` you just get `Some`. I'm not sure what would be the optimal display for enum variants in this case. Co-authored-by: Ville Penttinen <[email protected]>
| * Use ast::Name::text() instead of name().syntax().text()Ville Penttinen2019-03-071-2/+2
| |
| * Fix EnumVariants not showing properly when hoveringVille Penttinen2019-03-071-2/+56
|/ | | | This fixes documentation as well for EnumVariants
* Merge #948bors[bot]2019-03-071-6/+11
|\ | | | | | | | | | | | | | | | | | | | | | | 948: Fix test_missing_module_code_action_in_json_project on Windows r=matklad a=vipentti The test would fail on Windows due to the paths not being properly escaped for JSON. In addition adds extra braces around the fn main to actually introduce braces in the file. Co-authored-by: Ville Penttinen <[email protected]>
| * Fix test_missing_module_code_action_in_json_project on WindowsVille Penttinen2019-03-071-6/+11
| | | | | | | | | | | | | | | | The test would fail on Windows due to the paths not being properly escaped for JSON. In addition adds extra braces around the fn main to actually introduce braces in the file.
* | Merge #949bors[bot]2019-03-072-2/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | 949: Specify derive feature for serde r=matklad a=pcpthm `ra_project_model` build will fail when no dependencies are using this feature. Reproduced by creating a crate depending on `ra_batch`. Co-authored-by: pcpthm <[email protected]>
| * | Specify derive feature for serdepcpthm2019-03-072-2/+2
|/ /
* | Merge #944bors[bot]2019-03-074-26/+190
|\ \ | |/ |/| | | | | | | | | | | 944: Add support for goto definition and hover on Self r=matklad a=vipentti This fixes #943 Co-authored-by: Ville Penttinen <[email protected]>
| * Add support for goto definition and hover on SelfVille Penttinen2019-03-074-26/+190
|/ | | | This fixes #943
* Merge #946bors[bot]2019-03-073-22/+21
|\ | | | | | | | | | | | | | | 946: when loading workspace, say how many packages were loaded r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * when loading workspace, say how many packages were loadedAleksey Kladov2019-03-073-22/+21
|/ | | | | this should help to debug configuration issues, when you see `0 packages loaded` or something like that.
* Merge #939bors[bot]2019-03-079-93/+311
|\ | | | | | | | | | | | | | | | | | | | | 939: Initial implementation of project-lock.json. r=davidtwco a=davidtwco Fixes #792. This PR adds a initial implementation of project-lock.json, a build system agnostic method of specifying the crate graph and roots. Co-authored-by: David Wood <[email protected]>
| * Remove rust-project.json test w/ dependencies.David Wood2019-03-071-43/+0
| |
| * Add test demonstrating logic for handling deps.David Wood2019-03-071-1/+44
| |
| * Rename and change `add_roots` to return a `Vec`.David Wood2019-03-073-3/+8
| |
| * Initial implementation of project-lock.json.David Wood2019-03-079-93/+306
| | | | | | | | | | This commit adds a initial implementation of project-lock.json, a build system agnostic method of specifying the crate graph and roots.
* | Merge #942bors[bot]2019-03-072-4/+46
|\ \ | |/ |/| | | | | | | | | | | 942: Hover for associated items in patterns r=matklad a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * Use source mapkjeremy2019-03-062-15/+10
| |
| * Hover for associated items in patternskjeremy2019-03-063-9/+56
|/
* Merge #933bors[bot]2019-03-062-12/+45
|\ | | | | | | | | | | | | | | | | | | 933: Check installed extension r=matklad a=c410-f3r Fixes #918. Edit: Windows encoding for Unicode is UTF-16 so String::from_utf8 will probably fail unless `Vec<u8>` is already UTF-8 somehow, which I don't know for sure. Co-authored-by: Caio <[email protected]>
| * Check installed extensionCaio2019-03-052-12/+45
| |
* | Merge #941bors[bot]2019-03-061-10/+10
|\ \ | | | | | | | | | | | | | | | | | | | | | 941: :arrow_up: vfs r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | :arrow_up: vfsAleksey Kladov2019-03-061-108/+108
| | |
* | | Merge #940bors[bot]2019-03-0610-54/+110
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 940: Show workspace loaded notification r=matklad a=vipentti This fixes #935 This adds support for more `InitializationOptions` which are provided by the client. Co-authored-by: Ville Penttinen <[email protected]>
| * | | Add showWorkspaceLoadedNotification to vscode clientVille Penttinen2019-03-068-22/+78
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows users to control whether or not they want to see the "workspace loaded" notification. This is done on the server side using InitializationOptions which are provided by the client. By default show_workspace_loaded is true, meaning the notification is sent.
| * | | Rename feedback to show_messageVille Penttinen2019-03-053-13/+13
| | | |
| * | | Remove InternalFeedbackVille Penttinen2019-03-051-7/+0
| | | |
| * | | Send an actual ShowMessage instead of InternalFeedback in feedback()Ville Penttinen2019-03-053-20/+27
|/ / / | | | | | | | | | | | | | | | | | | This now allows us to send a notification that can be shown in the UI when the workspace has been loaded. Additionally this removes the need for internal_mode flag.
* | | Merge #938bors[bot]2019-03-056-114/+114
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 938: Updates r=matklad a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * | Updateskjeremy2019-03-056-114/+114
|/ /
* | Merge #936bors[bot]2019-03-051-1/+8
|\ \ | | | | | | | | | | | | | | | | | | | | | 936: dont produce giant debug dumps r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | dont produce giant debug dumpsAleksey Kladov2019-03-051-1/+8
|/ /
* | Merge #934bors[bot]2019-03-053-8/+32
|\ \ | | | | | | | | | | | | | | | | | | | | | 934: show message in client's UI if workspace fails to load r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>