aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge #9312bors[bot]2021-06-1712-171/+242
|\ | | | | | | | | | | | | | | 9312: internal: Move out item specific completion tests r=Veykril a=Veykril bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * Correct completions in items testsLukas Wirth2021-06-175-24/+57
| |
| * Move item specific completion testsLukas Wirth2021-06-176-71/+111
| |
| * Less filtering in completion testsLukas Wirth2021-06-177-98/+96
|/
* Merge #9310bors[bot]2021-06-1710-255/+299
|\ | | | | | | | | | | | | | | 9310: internal: Refine and test UseTree completions r=Veykril a=Veykril bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * Fix incorrect completions in empty braced use statementLukas Wirth2021-06-173-5/+18
| |
| * Refine `self`, `super` and `crate` completion in use pathsLukas Wirth2021-06-179-54/+47
| |
| * Move out and rewrite UseTree completion testsLukas Wirth2021-06-175-224/+262
| |
* | Merge #9308bors[bot]2021-06-172-13/+60
|\ \ | | | | | | | | | | | | | | | | | | | | | 9308: fix: Create modules in correct directory for nested modules in move_module assist r=Veykril a=Veykril bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | Create modules in correct directory for nested modules in move_module assistLukas Wirth2021-06-172-13/+60
| |/
* | Merge #9307bors[bot]2021-06-172-178/+37
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 9307: internal: switch some tests to minicore r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | internal: remove dead codeAleksey Kladov2021-06-171-24/+0
| | |
| * | internal: switch some tests to minicoreAleksey Kladov2021-06-172-154/+37
| | |
* | | Merge #9306bors[bot]2021-06-174-70/+80
|\| | | | | | | | | | | | | | | | | | | | | | | | | | 9306: internal: minimize minicore r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | internal: minimize minicoreAleksey Kladov2021-06-173-59/+36
| | | | | | | | | | | | | | | | | | We want to keep minicore small, so let's split out iterator adapters and sources into a separate `iterators` region, and use them only when needed.
| * | internal: switch some tests to minicoreAleksey Kladov2021-06-173-17/+50
|/ /
* | Merge #9304bors[bot]2021-06-166-159/+212
|\ \ | |/ |/| | | | | | | | | | | | | 9304: internal: cleanup tests r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * internal: cleanup testsAleksey Kladov2021-06-162-72/+140
| |
| * internal: switch some tests to minicoreAleksey Kladov2021-06-162-15/+8
| |
| * internal: add iterator to minicoreAleksey Kladov2021-06-162-39/+38
| |
| * internal: switch some tests to minicoreAleksey Kladov2021-06-161-15/+12
| |
| * internal: switch some tests to minicoreAleksey Kladov2021-06-161-19/+15
| |
* | Merge #9301bors[bot]2021-06-1626-414/+430
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | 9301: internal: Start refactoring ide_completion tests r=Veykril a=Veykril Our current completion test infra resovles around usually just checking a specific `CompletionKind` which is suboptimal. We only see what we want to see in tests with this causing us to miss a lot of incorrect completions we are doing. Instead we should test for different cursor locations for all kinds(sans the magic kind maybe? not sure yet). This way we will also see potential duplicate completions that merely different in their kind. Also since most completion submodules complete things in tests of other modules due to the tests overlapping it makes more sense to group these tests differently which implies moving them to a new module. Exceptions for this might be stuff like attribute completion as these cannot currently interfere. I only wrote a few tests to check for completions in `ItemList` position so far and I already found a few incorrect/irrelevant completions as these haven't been tested properly due to them being hidden by the `CompletionKind` filtering. I think `CompletionKind` doesn't really seem to be beneficial to me as to I can't think of a occasion where we would want to only check a specific completion kind. Co-authored-by: Lukas Wirth <[email protected]>
| * Move test_utils into tests moduleLukas Wirth2021-06-1626-238/+194
| |
| * Don't complete paths after attributesLukas Wirth2021-06-167-135/+123
| |
| * Don't show incorrect completions after unsafe or visiblity nodeLukas Wirth2021-06-164-42/+38
| |
| * Don't complete visibility accessors after existing onesLukas Wirth2021-06-164-3/+24
| |
| * Start refactoring ide_completion testsLukas Wirth2021-06-169-247/+302
|/
* Merge #9302bors[bot]2021-06-161-7/+67
|\ | | | | | | | | | | | | | | 9302: internal: Add builtin derives to attribute completion fixtures r=Veykril a=Veykril bors R+ Co-authored-by: Lukas Wirth <[email protected]>
| * Don't complete already used derive attributesLukas Wirth2021-06-161-20/+20
| |
| * Add builtin derives to attribute completion fixturesLukas Wirth2021-06-161-7/+67
| |
* | Merge #9258bors[bot]2021-06-162-8/+10
|\ \ | | | | | | | | | | | | | | | | | | | | | 9258: minor: Give `ImportPrefix` variants better config names r=matklad a=Veykril I feel like `crate` and `self` work better than `by_crate` and `by_self`. The only reason for the current names were that `Self` doesn't work for the variant name on the rust side so I forgot about setting proper config names on serde layer. Co-authored-by: Lukas Wirth <[email protected]>
| * | Give ImportPrefix variants better config namesLukas Wirth2021-06-132-8/+10
| | |
* | | Merge #9300bors[bot]2021-06-163-66/+34
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | 9300: minor: Simplify r=Veykril a=Veykril bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | SimplifyLukas Wirth2021-06-163-66/+34
|/ /
* | Merge #9299bors[bot]2021-06-166-71/+94
|\ \ | | | | | | | | | | | | | | | | | | | | | 9299: minor: Filter out non-type completions in the respective completions modules instead r=Veykril a=Veykril bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | Filter out non-type completions in the respective completions modules insteadLukas Wirth2021-06-166-71/+94
| | |
* | | Merge #9297bors[bot]2021-06-167-345/+307
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9297: internal: add fn to minicore r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | internal: add fn to minicoreAleksey Kladov2021-06-166-310/+258
| | | |
| * | | internal: add deref_mut to minicoreAleksey Kladov2021-06-163-35/+49
|/ / /
| | |
| \ \
*-. \ \ Merge #9267 #9279bors[bot]2021-06-155-21/+24
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9267: fix: Code: update the LSP server without asking r=matklad a=lnicola Most LSP extensions seem to do the same thing, and this is causing some confusion for users who don't notice the update prompt before Code hides it. 9279: minor: Document installation via Homebrew r=matklad a=Svetlitski `rust-analyzer` can be installed via [Homebrew](https://brew.sh) (AKA`brew`) on macOS. I've added instructions on how to do so to the documentation. Additionally, I added a `.gitignore` rule to ignore the HTML documentation produced by `asciidoctor manual.adoc` so that it is not accidentally checked into `git`. Co-authored-by: LaurenČ›iu Nicola <[email protected]> Co-authored-by: Kevin Svetlitski <[email protected]> Co-authored-by: Aleksey Kladov <[email protected]>
| | * | | Update docs/user/manual.adocAleksey Kladov2021-06-151-1/+1
| | | | |
| | * | | Document installation via Homebrew; Add gitignore entry for asciidoctor outputKevin Svetlitski2021-06-142-0/+10
| | | | |
| * | | | Consolidate the privacy notesLaurențiu Nicola2021-06-153-20/+13
| | | | |
| * | | | Code: automatically update the LSP serverLaurențiu Nicola2021-06-142-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Most LSP extensions seem to do the same thing, and this is causing some confusion for users who don't notice the update prompt before Code hides it.
* | | | | Merge #9293bors[bot]2021-06-151-3/+4
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9293: fix: Show diagnostic fixes before assists r=matklad a=lnicola Closes #9212 Co-authored-by: LaurenČ›iu Nicola <[email protected]>
| * | | | | Show diagnostic fixes before assistsLaurențiu Nicola2021-06-151-3/+4
| | | | | |
* | | | | | Merge #9294bors[bot]2021-06-1513-1027/+1149
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9294: internal: introduce minicore -- a subset of libcore for testing r=matklad a=matklad Clearly, we need one more fixed point iteration loop! Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | | internal: switch some tests to minicoreAleksey Kladov2021-06-152-18/+9
| | | | | | |
| * | | | | | internal: add result to minicoreAleksey Kladov2021-06-152-21/+18
| | | | | | |