aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Remove debug printAleksey Kladov2020-12-101-7/+5
|
* Merge #6798bors[bot]2020-12-105-446/+43
|\ | | | | | | | | | | | | | | | | | | 6798: Ignore extern items in incorrect-case check r=jonas-schievink a=jonas-schievink Fixes https://github.com/rust-analyzer/rust-analyzer/issues/6736 bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * Use mark/hitJonas Schievink2020-12-101-0/+7
| |
| * Remove item tree testsJonas Schievink2020-12-102-441/+0
| | | | | | | | | | They were useful during initial development of the item tree, but now just cause churn
| * Ignore extern items in incorrect-case checkJonas Schievink2020-12-104-5/+36
| |
* | Merge #6801bors[bot]2020-12-104-16/+12
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 6801: Minor, more orthogonal code r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | Minor, more orthogonal codeAleksey Kladov2020-12-104-16/+12
| | | | | | | | | | | | It's better to accept things as arguments rather than store them.
* | | Merge #6797bors[bot]2020-12-1010-43/+44
|\| | | |/ |/| | | | | | | | | | | | | 6797: Normalize spelling to American English r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * Normalize spelling to American EnglishAleksey Kladov2020-12-1010-43/+44
|/
* Merge #6796bors[bot]2020-12-101-4/+4
|\ | | | | | | | | | | | | | | | | | | 6796: Fix FileId computation in IncorrectCase fix r=jonas-schievink a=jonas-schievink Fixes the sub-issue mentioned in https://github.com/rust-analyzer/rust-analyzer/issues/4058#issuecomment-741919659 bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * Fix FileId computation in IncorrectCase fixJonas Schievink2020-12-101-4/+4
|/
* Merge #6795bors[bot]2020-12-101-0/+6
|\ | | | | | | | | | | | | | | | | | | 6795: format_args: handle key-value arguments r=jonas-schievink a=jonas-schievink fixes https://github.com/rust-analyzer/rust-analyzer/issues/6728 bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * format_args: handle key-value argumentsJonas Schievink2020-12-101-0/+6
|/
* Merge #6790bors[bot]2020-12-092-0/+13
|\ | | | | | | | | | | | | | | 6790: Send a CodeLensRefresh when we reload r=kjeremy a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * Send a CodeLensRefresh when we reloadkjeremy2020-12-092-0/+13
| |
* | Merge #6789bors[bot]2020-12-091-1/+1
|\ \ | |/ |/| | | | | | | | | | | 6789: Fix a typo in manual.adoc r=lnicola a=cuviper Co-authored-by: Josh Stone <[email protected]>
| * Fix a typo in manual.adocJosh Stone2020-12-091-1/+1
|/
* Merge #6786bors[bot]2020-12-091-20/+23
|\ | | | | | | | | | | | | | | | | | | 6786: Implement HirDisplay for FnSig r=jonas-schievink a=jonas-schievink This could be useful for diagnostics, but isn't used right now bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * Implement HirDisplay for FnSigJonas Schievink2020-12-091-20/+23
|/ | | | This could be useful for diagnostics, but isn't used right now
* Merge #6785bors[bot]2020-12-093-8/+19
|\ | | | | | | | | | | | | | | | | | | | | | | 6785: Fix "no value set for FileTextQuery(FileId(..))" r=jonas-schievink a=jonas-schievink Fixes https://github.com/rust-analyzer/rust-analyzer/issues/6622 Let's hope I got it right this time, but I feel like I slowly begin to understand the main loop logic. bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * Fix "no value set for FileTextQuery(FileId(..))"Jonas Schievink2020-12-093-8/+19
| |
* | Merge #6784bors[bot]2020-12-0916-72/+130
|\ \ | |/ |/| | | | | | | | | | | | | 6784: Introduce anchored_path r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * Introduce anchored_pathAleksey Kladov2020-12-0913-65/+79
| | | | | | | | | | They allow to represent paths like `#[path = "C:\path.rs"] mod foo;` in a lossless cross-platform & network-transparent way.
| * .Aleksey Kladov2020-12-094-8/+52
|/
* Merge #6783bors[bot]2020-12-091-2/+1
|\ | | | | | | | | | | | | | | | | | | 6783: Avoid nesting Subtrees when parsing TokenStream r=jonas-schievink a=jonas-schievink Fixes https://github.com/rust-analyzer/rust-analyzer/issues/6744 bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * Avoid nesting Subtrees when parsing TokenStreamJonas Schievink2020-12-091-2/+1
|/
* Merge #6771bors[bot]2020-12-093-5/+117
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6771: Properly attach inner attributes in Attrs::new r=matklad a=Veykril Properly attach inner and outer attributes to the things they actually belong to in the HIR. ~~I can add some tests for this if wanted once I know where to put them/how to test for this.~~ Put some tests into `hover.rs`. So the following snippet ```rust mod foo { //! Hello } ``` now shows `Hello` on hover 🎉 Fixes #2148 Co-authored-by: Lukas Wirth <[email protected]>
| * Test inner and outer doc comments in hoverLukas Wirth2020-12-091-0/+62
| |
| * Properly fetch inner and outer docs on hir-levelLukas Wirth2020-12-092-17/+33
| |
| * Properly fetch inner and outer attributes on hir-levelLukas Wirth2020-12-081-5/+39
| |
* | Merge #6776bors[bot]2020-12-094-19/+174
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 6776: Include config into the manual r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | Include config into the manualAleksey Kladov2020-12-094-19/+174
| | |
* | | Merge #6775bors[bot]2020-12-091-2/+3
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 6775: Fix MissingFields DiagnosticFix trigger range r=lnicola a=Veykril Fixes #5950 Co-authored-by: Lukas Wirth <[email protected]>
| * | Fix MissingFields DiagnosticFix trigger rangeLukas Wirth2020-12-091-2/+3
| |/
* | Merge #6774bors[bot]2020-12-091-2/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | 6774: Increment recursion count in Expander only on success r=lnicola a=Veykril Fixes #6764 Co-authored-by: Lukas Wirth <[email protected]>
| * | Properly decrement recursion count in ExpanderLukas Wirth2020-12-091-2/+2
| |/
* | Merge #6770bors[bot]2020-12-081-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | 6770: Update usage string to include code-exploration client. r=lnicola a=derdaele Support for `code-exploration` client was added #6766 in but the usage string wasn't updated. Co-authored-by: Jérémy <[email protected]>
| * | Update usage string to include code-exploration client.Jérémy2020-12-081-1/+1
| | |
* | | Merge #6768bors[bot]2020-12-081-8/+34
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | 6768: Fix `concat!` with integer literals r=jonas-schievink a=jonas-schievink Addresses https://github.com/rust-analyzer/rust-analyzer/issues/6747#issuecomment-740778402 bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | Fix `concat!` with integer literalsJonas Schievink2020-12-081-8/+34
| | |
* | | Merge #6766bors[bot]2020-12-081-6/+14
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6766: Add client install support for `code-exploration` builds. r=lnicola a=derdaele VSCode has a feature to install the command to the PATH. <img width="640" alt="Capture d’écran 2020-12-08 à 19 25 43" src="https://user-images.githubusercontent.com/16373039/101525141-2e013300-398b-11eb-8d07-60a92ae9587c.png"> `code-exploration` is the command name for the ARM64 experimental build. As of today, this is the only build running natively on Apple Silicon. See _ARM64_ Experimental in https://code.visualstudio.com/insiders/#osx. The `-exploration` prefix seems pretty undocumented, my understanding of it is that it is an insider-like version that uses a different electron version (in this case, maybe the election version that was recently ported to Apple Silicon?). Co-authored-by: Jérémy <[email protected]>
| * | Add client install support for `code-exploration` build.Jérémy2020-12-081-6/+14
| |/
* | Merge #6767bors[bot]2020-12-081-1/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | 6767: Better error when a proc macro panics r=jonas-schievink a=jonas-schievink bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | Better error when a proc macro panicsJonas Schievink2020-12-081-1/+2
|/ /
* | Merge #6761bors[bot]2020-12-087-432/+659
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6761: Make config.rs a single source of truth for configuration. r=matklad a=matklad Configuration is editor-independent. For this reason, we pick JSON-schema as the repr of the source of truth. We do specify it using rust-macros and some quick&dirty hackery though. The idea for syncing truth with package.json is to just do that manually, but there's a test to check that they are actually synced. I'll add something like `rust-analyzer --config-schema` in a follow-up commit. Co-authored-by: Aleksey Kladov <[email protected]>
| * | Make config.rs a single source of truth for configuration.Aleksey Kladov2020-12-087-432/+659
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Configuration is editor-independent. For this reason, we pick JSON-schema as the repr of the source of truth. We do specify it using rust-macros and some quick&dirty hackery though. The idea for syncing truth with package.json is to just do that manually, but there's a test to check that they are actually synced. There's CLI to print config's json schema: $ rust-analyzer --print-config-schema We go with a CLI rather than LSP request/response to make it easier to incorporate the thing into extension's static config. This is roughtly how we put the thing in package.json.
* | Merge #6765bors[bot]2020-12-086-98/+89
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6765: Fix file range computation in macros r=jonas-schievink a=jonas-schievink This also aligns the diagnostics behavior of `TestDB` with the one from the real IDE (by making the logic from `semantics.rs` a method on `InFile<&SyntaxNode>`), which makes bugs like this easier to find. This should fix the misplaced diagnostics seen in https://github.com/rust-analyzer/rust-analyzer/issues/6747 and other issues. bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | Fix logic for determining macro callsJonas Schievink2020-12-081-9/+7
| | | | | | | | | | | | | | | | | | I believe this currently goes back all the way to the initial user-written call, but that seems better than the current broken behavior.
| * | Use `original_file_range` in `TestDB`Jonas Schievink2020-12-081-9/+3
| | |
| * | Make `original_range` a method on `InFile<&SyntaxNode>`Jonas Schievink2020-12-085-89/+88
|/ /