aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge #265bors[bot]2018-12-084-24/+59
|\ | | | | | | | | | | | | | | | | 265: Refactor symbol resolve API r=matklad a=matklad Introduce ReferenceResolution to avoid nesting to many non-nominal types. Co-authored-by: Aleksey Kladov <[email protected]>
| * Refactor symbol resolve APIAleksey Kladov2018-12-084-24/+59
| | | | | | | | | | Introduce ReferenceResolution to avoid nesting to many non-nominal types.
* | Merge #263bors[bot]2018-12-0817-22/+28
|\| | | | | | | | | | | | | | | 263: New modules r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * account for new layout when collecting testsAleksey Kladov2018-12-081-2/+8
| |
| * grand module renameAleksey Kladov2018-12-0815-0/+0
| |
| * resolve 2018 style modulesAleksey Kladov2018-12-081-20/+20
| |
* | Merge #264bors[bot]2018-12-081-1/+20
|\ \ | |/ |/| | | | | | | | | | | 264: check for empty range when extending in comment r=matklad a=vemoo fix for #140 Co-authored-by: Bernardo <[email protected]>
| * check for empty range when extending in commentBernardo2018-12-081-1/+20
| |
* | reformatAleksey Kladov2018-12-082-2/+3
| |
* | Add a better text for hover and stop duplicating work done in ↵DJMcNab2018-12-087-30/+98
| | | | | | | | approximatelly_resolve_symbol
* | Fix typo in commentDJMcNab2018-12-081-10/+15
| |
* | Clarify and correct comment about multi_byte_tokensDJMcNab2018-12-081-2/+2
| |
* | Fix the range of a hover request to be more in line with prior artDJMcNab2018-12-081-1/+8
|/
* Merge #261bors[bot]2018-12-066-19/+177
|\ | | | | | | | | | | | | | | 261: Add heavy test for code actions r=matklad a=flodiebold Here's the test for the code actions; I didn't find anything fitting on crates.io ([assert-json-diff](https://crates.io/crates/assert-json-diff) looks kind of nice, but doesn't have anything like the wildcards), so I copied the cargo code as you suggested. Co-authored-by: Florian Diebold <[email protected]>
| * Add test for code actionsFlorian Diebold2018-12-061-1/+60
| |
| * Use json comparison code from cargo for heavy testsFlorian Diebold2018-12-066-18/+117
|/
* Merge #260bors[bot]2018-12-0627-180/+82
|\ | | | | | | | | | | | | | | 260: Modernize r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * modernize even moreAleksey Kladov2018-12-0612-70/+29
| |
| * modernize moreAleksey Kladov2018-12-0610-56/+42
| |
| * drop extern cratesAleksey Kladov2018-12-063-26/+0
| |
| * modernize some filesAleksey Kladov2018-12-063-10/+4
| |
| * modernize some codeAleksey Kladov2018-12-063-18/+7
| |
* | Merge #259bors[bot]2018-12-061-1/+1
|\ \ | |/ |/| | | | | | | | | | | 259: dedupe testing & formatting toolchains r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * dedupe testing & formatting toolchainsAleksey Kladov2018-12-061-1/+1
|/
* Merge #256bors[bot]2018-12-0611-1/+467
|\ | | | | | | | | | | | | | | 256: Improve/add use_item documentation r=matklad a=DJMcNab Adds some documentation to use_item explaining all code paths (use imports are hard, especially with the ongoing discussion of anchored v. uniform paths - see https://github.com/rust-lang/rust/issues/55618 for what appears to be the latest developments) Co-authored-by: DJMcNab <[email protected]>
| * Update use path testDJMcNab2018-12-052-74/+2
| |
| * Improve/add the use_item documentationDJMcNab2018-12-0511-1/+539
| |
* | Merge #258bors[bot]2018-12-062-2/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | 258: :arrow_up: 1.31.0 :tada: r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | :arrow_up: 1.31.0 :tada:Aleksey Kladov2018-12-062-2/+2
|/ /
* | Merge #253bors[bot]2018-12-051-2/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 253: Fix diagnostic fixes showing up everywhere r=matklad a=flodiebold The LSP code action request always returned the fixes for all diagnostics anywhere in the file, because of a shadowed variable. There's no test yet; I wasn't sure where to add it. I tried adding one in `heavy_tests`, but that's a bit uncomfortable because the code action response contains the (random) file paths. I could make it work, but wanted to ask beforehand what you think. Co-authored-by: Florian Diebold <[email protected]>
| * | Fix diagnostic fixes showing up everywhereFlorian Diebold2018-12-021-2/+2
| | | | | | | | | | | | | | | The LSP code action request always returned the fixes for all diagnostics anywhere in the file, because of a shadowed variable.
* | | Merge #257bors[bot]2018-12-051-1/+1
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | 257: Fix a copy and pasting typo in lexer/ptr.rs r=matklad a=DJMcNab Co-authored-by: DJMcNab <[email protected]>
| * | Fix a copy and pasting typoDJMcNab2018-12-051-1/+1
|/ /
* | First step towards crate depsAleksey Kladov2018-12-051-8/+35
| |
* | make stuff privateAleksey Kladov2018-12-051-1/+1
| |
* | Merge #255bors[bot]2018-12-057-140/+125
|\ \ | | | | | | | | | | | | | | | | | | | | | 255: Binders r=matklad a=matklad Binding sources to hir is a fuzzy operation, so let's move it to a special enclave in the source code. Co-authored-by: Aleksey Kladov <[email protected]>
| * | make stuff privateAleksey Kladov2018-12-052-13/+8
| | |
| * | move fuzzy source binding to a separete modeAleksey Kladov2018-12-057-131/+121
|/ /
* | restore index-based gotodefAleksey Kladov2018-12-052-5/+21
| |
* | Merge #254bors[bot]2018-12-0412-160/+186
|\ \ | | | | | | | | | | | | | | | | | | | | | 254: Defids r=matklad a=matklad Fleshing out DefIds some more Co-authored-by: Aleksey Kladov <[email protected]>
| * | minorAleksey Kladov2018-12-042-7/+17
| | |
| * | Add functions to DefIdAleksey Kladov2018-12-0410-83/+90
| | |
| * | module-scoped deflocAleksey Kladov2018-12-043-24/+43
| | |
| * | ModuleSource is ItemSourceAleksey Kladov2018-12-044-51/+40
| | |
| * | include file itself in SourceFileItemsAleksey Kladov2018-12-041-1/+2
|/ /
* | Merge #250bors[bot]2018-12-022-5/+11
|\ \ | |/ |/| | | | | | | | | | | | | 250: Improve the suggestion for test functions r=DJMcNab a=DJMcNab I haven't fully updated the previous commented out test - I don't know why it was commented out so some clarification would be welcome. Co-authored-by: Daniel McNab <[email protected]> Co-authored-by: DJMcNab <[email protected]>
| * Fix formattingDJMcNab2018-12-021-4/+4
| |
| * Format completion text properlyDaniel McNab2018-11-281-2/+5
| |
| * Add tfn lookup and remove test prefixDaniel McNab2018-11-282-3/+3
| |
| * Change the body and name of the test function completionDaniel McNab2018-11-282-4/+7
| |