aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | Merge #8595bors[bot]2021-04-225-17/+62
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8595: Diagnostic paths support specifying `remapPrefix` r=rickvanprim a=rickvanprim Currently VSCode Problem Matchers will resolve a path like `//foo_crate/src/main.rs` if `${workspaceFolder}/foo_crate/src/main.rs` exists. Presumably their behavior is functionally a string concatenation that would produce `${workspaceFolder///foo_crate/src/main.rs` and repeated path separators get ignored. This PR attempts to mimic this behavior by stripping any `Component::RootDir` from `file_name` before joining it to `workspace_root`, and then checking if the file exists. If it does, this path is used, and if not, the behavior falls through to the existing Rust path join behavior. Co-authored-by: James Leitch <[email protected]>
| * | | | | | | Code review feedback.James Leitch2021-04-215-12/+14
| | | | | | | |
| * | | | | | | Diagnostic Remap Path Prefixes added.James Leitch2021-04-215-37/+53
| | | | | | | |
| * | | | | | | Diagnostic paths attempt to use VSCode's path join behavior before ↵James Leitch2021-04-211-3/+30
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | defaulting to Rust's path join behavior.
* | | | | | | Merge #8570bors[bot]2021-04-213-22/+48
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8570: Flycheck tries to parse both Cargo and Rustc messages. r=rickvanprim a=rickvanprim This change allows non-Cargo build systems to be used for Flycheck provided they call `rustc` with `--error-format=json` and emit those JSON messages to `stdout`. Co-authored-by: James Leitch <[email protected]>
| * | | | | | | Code review feedback.James Leitch2021-04-191-15/+16
| | | | | | | |
| * | | | | | | Flycheck tries to parse both Cargo and Rustc messages.James Leitch2021-04-193-22/+47
| | | | | | | |
* | | | | | | | Merge #8616bors[bot]2021-04-213-20/+20
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8616: minor: Bump deps r=lnicola a=lnicola bors r+ Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | | | | | Bump other depsLaurențiu Nicola2021-04-211-6/+6
| | | | | | | | |
| * | | | | | | | Bump chalkLaurențiu Nicola2021-04-212-11/+11
| | | | | | | | |
| * | | | | | | | Bump rustc_lexerLaurențiu Nicola2021-04-212-3/+3
| | |_|_|_|/ / / | |/| | | | | |
* | | | | | | | Merge #8615bors[bot]2021-04-211-7/+0
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8615: skip: Remove `ItemTree::all_inner_items` r=jonas-schievink a=jonas-schievink It's unused. bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | | | | | | Remove `ItemTree::all_inner_items`Jonas Schievink2021-04-211-7/+0
|/ / / / / / /
* | | | | | | Merge #8613bors[bot]2021-04-213-0/+46
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8613: skip: Add failing local items test r=jonas-schievink a=jonas-schievink This is currently broken due to the way ItemTree stores inner items. This test serves as a reminder until this gets fixed. bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | | | | | | Add failing local items testJonas Schievink2021-04-213-0/+46
|/ / / / / / /
* | | | | | | Merge #8611bors[bot]2021-04-212-23/+207
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8611: Add support for fill match arms of boolean values r=flodiebold a=komonad - Add support for boolean inside tuple closes #8593 Co-authored-by: Comonad <[email protected]>
| * | | | | | Add support for fill match arms of boolean valuesComonad2021-04-212-23/+207
|/ / / / / / | | | | | | | | | | | | | | | | | | - Add support for boolean inside tuple
* | | | | | Merge #8609bors[bot]2021-04-213-8/+57
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8609: Parse outer atttributes for RecordPatField r=Veykril a=Veykril Fixes #8608 bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | | | | | Parse outer atttributes for RecordPatFieldLukas Wirth2021-04-213-8/+57
|/ / / / / /
* | | | | | Merge #8606bors[bot]2021-04-202-2/+13
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8606: fix: no more Registering progress handler for token rustAnalyzer/Ind… r=jonas-schievink a=matklad …exing failed. closes #8509 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | fix: no more Registering progress handler for token rustAnalyzer/Indexing ↵Aleksey Kladov2021-04-202-2/+13
|/ / / / / | | | | | | | | | | | | | | | failed.
* | | | | Merge #8600bors[bot]2021-04-205-65/+330
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8600: fix: no longer get stuck on windows r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | fix: no longer get stuck on windowsAleksey Kladov2021-04-205-65/+330
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | reading both stdout & stderr is a common gotcha, you need to drain them concurrently to avoid deadlocks. Not sure why I didn't do the right thing from the start. Seems like I assumed the stderr is short? That's not the case when cargo spams `compiling xyz` messages
* | | | | | Merge #8602bors[bot]2021-04-201-28/+45
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8602: Fix panic in `replace_derive_with_manual_impl` r=jonas-schievink a=jonas-schievink bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | | | | | Follow testing style guideJonas Schievink2021-04-201-28/+28
| | | | | | |
| * | | | | | Fix panic in `replace_derive_with_manual_impl`Jonas Schievink2021-04-201-0/+17
| |/ / / / /
* | | | | | Merge #8601bors[bot]2021-04-201-38/+185
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8601: Trigger "Inline variable" assist when on a use of the variable r=jonas-schievink a=jonas-schievink Fixes https://github.com/rust-analyzer/rust-analyzer/issues/8530 bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | | | | "Inline variable" when on a use of the variableJonas Schievink2021-04-201-38/+185
| | |/ / / | |/| | |
* | | | | Merge #8598bors[bot]2021-04-201-26/+48
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8598: minor: fix ugly line wrapping in the rendered manual r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | minor: fix ugly line wrapping in the rendered manualAleksey Kladov2021-04-201-26/+48
| | | | |
| | | | * Don't filter equal nodes in reorder assistsLukas Wirth2021-04-212-10/+6
| | | | |
| | | | * Add cov_marks to insert_use testsLukas Wirth2021-04-207-42/+77
| | | | |
| | | | * Rewrite extract_struct_from_enum_variant assistLukas Wirth2021-04-201-85/+91
| | | | |
| | | | * Remove SyntaxRewriter usage in insert_use in favor of tedLukas Wirth2021-04-208-243/+185
| | | | |
| | | | * Replace SyntaxRewriter usage with ted in reorder_impl assistLukas Wirth2021-04-201-9/+13
| |_|_|/ |/| | |
* | | | Merge #8588bors[bot]2021-04-191-0/+11
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8588: internal: Add guidelines for release notes PR descriptions r=matklad a=lnicola Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | internal: Add guidelines for release notes PR descriptionsLaurențiu Nicola2021-04-191-0/+11
| | | | |
* | | | | Merge #8586bors[bot]2021-04-192-23/+18
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8586: Replace SyntaxRewriter usage with ted in eager::eager_macro_recur r=Veykril a=Veykril Co-authored-by: Lukas Wirth <[email protected]>
| * | | | | Replace SyntaxRewriter with ted in exppand_macro::expand_macro_recurLukas Wirth2021-04-191-14/+9
| | | | | |
| * | | | | Remove SyntaxRewriter usage in eager::eager_macro_recurLukas Wirth2021-04-191-9/+9
| | | | | |
* | | | | | Merge #8587bors[bot]2021-04-192-5/+91
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8587: Fix some find_path bugs around inner items r=jonas-schievink a=jonas-schievink Fixes https://github.com/rust-analyzer/rust-analyzer/issues/8519 bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | | | | Add autoimport test with inner itemsJonas Schievink2021-04-191-0/+33
| | | | | |
| * | | | | Fix some find_path bugs around inner itemsJonas Schievink2021-04-191-5/+58
|/ / / / /
* | | | | Merge #8584bors[bot]2021-04-191-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8584: internal: fix slightly broken test r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | internal: fix slightly broken testAleksey Kladov2021-04-191-1/+1
| | | | | |
* | | | | | Merge #8583bors[bot]2021-04-191-23/+14
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8583: Simplify r=Veykril a=Veykril bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | | | | SimplifyLukas Wirth2021-04-191-23/+14
| |/ / / /
* | | | | Merge #8582bors[bot]2021-04-194-5/+5
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8582: Fix typo: comparision -> comparison r=kjeremy a=stanciuadrian Closes #8576 Co-authored-by: Adrian Stanciu <[email protected]>
| * | | | | Fix typo: comparision -> comparisonAdrian Stanciu2021-04-194-5/+5
| |/ / / /
* | | | | Merge #8580bors[bot]2021-04-191-3/+8
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8580: Remove confusion around serverStatusNotification r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>