aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | Ignore unnamed trait fns and add more testsIgor Matuszewski2019-03-161-2/+70
| | | |
| * | | Simplify calculation of missing functionsIgor Matuszewski2019-03-161-9/+2
| | | | | | | | | | | | | | | | | | | | | | | | Asymptotically computing a set difference is faster but in the average case we won't have more than ~10 functions. Also prefer not using hash sets as these may yield nondeterministic results.
| * | | Properly support the case when the cursor is inside an empty block or outsideIgor Matuszewski2019-03-161-15/+44
| | | |
| * | | Redo indent calculation when adding missing impl membersIgor Matuszewski2019-03-161-9/+21
| | | |
| * | | Implement a simple working assistIgor Matuszewski2019-03-161-13/+59
| | | |
| * | | Calculate missing functions from impl bodyIgor Matuszewski2019-03-161-1/+62
| | | |
| * | | Add 'add_missing_impl_members' assist stubIgor Matuszewski2019-03-162-0/+43
|/ / /
* | | Merge #981bors[bot]2019-03-165-120/+157
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 981: Remove FnSig from FnDef type r=matklad a=flodiebold It doesn't need to be in there since it's just information from the def. Another step towards aligning Ty with Chalk's representation. Co-authored-by: Florian Diebold <[email protected]>
| * | | Remove FnSig from FnDef typeFlorian Diebold2019-03-164-21/+26
| | | | | | | | | | | | | | | | | | | | It doesn't need to be in there since it's just information from the def. Another step towards aligning Ty with Chalk's representation.
| * | | Refactor FnSig a bitFlorian Diebold2019-03-163-70/+104
| | | |
| * | | Some more Ty displaying cleanupFlorian Diebold2019-03-162-35/+33
|/ / /
* | | Merge #976bors[bot]2019-03-167-60/+136
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 976: Replace Display by a pretty printing trait for Ty r=matklad a=flodiebold This allows removing the names from Adt and FnDef (and more later), as a first step towards aligning more with chalk's Ty :) I may have gone a bit overboard with the definition of the PrettyPrint trait... Co-authored-by: Florian Diebold <[email protected]>
| * | | Replace Display by a pretty printing trait for TyFlorian Diebold2019-03-167-60/+136
| | | | | | | | | | | | | | | | | | | | This allows removing the names from Adt and FnDef (and more later), as a first step towards aligning more with chalk's Ty :)
* | | | Merge #978bors[bot]2019-03-151-16/+16
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | 978: Update some dependencies r=matklad a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * | | Update some dependencieskjeremy2019-03-151-16/+16
|/ / /
* | | Merge #977bors[bot]2019-03-151-5/+4
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 977: Fix ra-emacs-lsp for newest lsp-mode with native json support r=matklad a=flodiebold Co-authored-by: Florian Diebold <[email protected]>
| * | Fix ra-emacs-lsp for newest lsp-mode with native json supportFlorian Diebold2019-03-141-5/+4
|/ /
* | Merge #975bors[bot]2019-03-141-1/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | 975: sort navigations to make tests stable r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | sort navigations to make tests stableAleksey Kladov2019-03-141-1/+2
|/ /
* | Merge #974bors[bot]2019-03-143-1/+348
|\ \ | | | | | | | | | | | | | | | | | | | | | 974: Add MINUS token the PATTERN_FIRST token set r=matklad a=mchesser Fixes: #972 Co-authored-by: Michael Chesser <[email protected]>
| * | Add test for minus in inner patternMichael Chesser2019-03-142-0/+347
| | |
| * | Allow MINUS at the start of a pattern.Michael Chesser2019-03-141-1/+1
|/ /
* | Merge #973bors[bot]2019-03-141-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | 973: derive Hash for ra_hir::ModuleDef r=matklad a=pcpthm I wanted to use `HashSet` but it seems like `Hash` derive is missing for no reason. Co-authored-by: pcpthm <[email protected]>
| * | derive Hash for ra_hir::ModuleDefpcpthm2019-03-141-1/+1
|/ /
* | Merge #971bors[bot]2019-03-141-2/+3
|\ \ | | | | | | | | | | | | | | | | | | | | | 971: Point to compiler-team repo in the readme r=matklad a=detrumi The readme still links to the old repo, which redirects to the new one Co-authored-by: Wilco Kusee <[email protected]>
| * | Point to the compiler-team repo directlyWilco Kusee2019-03-131-2/+3
|/ /
* | Merge #970bors[bot]2019-03-131-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | 970: Fix typo r=matklad a=skade Co-authored-by: Florian Gilcher <[email protected]>
| * | Fix typoFlorian Gilcher2019-03-131-1/+1
|/ /
* | Merge #969bors[bot]2019-03-131-1/+2
|\ \ | |/ |/| | | | | | | | | | | 969: activate extension if Cargo.toml is present r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * activate extension if Cargo.toml is presentAleksey Kladov2019-03-131-1/+2
|/
* Merge #959bors[bot]2019-03-121-2/+16
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | 959: Retrieve current working directory from workspace r=matklad a=LDSpits This PR improves the way the language client retrieves the current working directory by using the VSCode workspace API to get the path to the currently open directory. If we find more than one directory we show a warning that "multi root workspaces are not supported yet" and pick the root path. Any feedback is appreciated 😄 fixes #945 Co-authored-by: Lucas Spits <[email protected]>
| * Applied code style of ``npm run fix``Lucas Spits2019-03-111-6/+4
| |
| * Fix typescript linting errorsLucas Spits2019-03-111-1/+1
| |
| * Merge branch 'master' of https://github.com/LDSpits/rust-analyzer into ↵Lucas Spits2019-03-1114-69/+174
| |\ | | | | | | | | | feature/workspace-directory
| * | Retrieve current directory from workspacesLucas Spits2019-03-111-2/+18
| | |
* | | Merge #958bors[bot]2019-03-121-0/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 958: LSP: Support EnumMember and Field r=matklad a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * | | 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]>