aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge #8317bors[bot]2021-04-235-7/+636
|\ | | | | | | | | | | | | | | 8317: Convert tuple struct to named struct assist r=Veykril a=unexge Closes https://github.com/rust-analyzer/rust-analyzer/issues/8192 Co-authored-by: unexge <[email protected]>
| * Add missing test case for "Convert to named struct" assistunexge2021-04-231-0/+40
| |
| * Stop iterating reference after made an edit in "Convert to named struct" assistunexge2021-04-231-2/+47
| |
| * Move reference editing logic into own function to make error handling more ↵unexge2021-04-221-68/+61
| | | | | | | | ease in "Convert to named struct" assist
| * Use multiple loops instead of `Iterator::chain` in `FindUsages`unexge2021-04-211-13/+22
| |
| * Add multi file test for "Convert to named struct" assistunexge2021-04-211-0/+33
| |
| * Remove `unwrap`s in "Convert to named struct" assistunexge2021-04-211-9/+19
| |
| * Add larger example for "Convert to named struct" assistunexge2021-04-212-8/+60
| |
| * Fix incorrectly replacing method calls in "Convert to named struct" assistunexge2021-04-211-17/+41
| |
| * Add convert tuple struct to named struct assistunexge2021-04-045-9/+397
| |
| * Allow including `Self` kw references to `FindUsages`unexge2021-04-041-4/+39
| |
* | Merge #8628bors[bot]2021-04-221-0/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 8628: minor: Add one more profiling span to add_dep r=lnicola a=lnicola CC #8623 bors r+ Co-authored-by: Laurențiu Nicola <[email protected]>
| * | Add one more profiling span to add_depLaurențiu Nicola2021-04-221-0/+1
| | |
* | | Merge #8627bors[bot]2021-04-224-3/+7
|\| | | | | | | | | | | | | | | | | | | | | | | 8627: Add profiling spans under cargo_to_crate_graph r=lnicola a=lnicola bors r+ Co-authored-by: Laurențiu Nicola <[email protected]>
| * | Add profiling spans under cargo_to_crate_graphLaurențiu Nicola2021-04-224-3/+7
| | |
* | | Merge #8605bors[bot]2021-04-223-43/+173
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8605: internal: Automatically categorize the changelog entries r=matklad a=lnicola Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | Automatically categorize the changelog entriesLaurențiu Nicola2021-04-203-43/+173
| | | |
* | | | Merge #8620bors[bot]2021-04-221-14/+22
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8620: Remove unnecessary braces for extracted block expression r=Veykril a=brandondong This change addresses the first bullet point of https://github.com/rust-analyzer/rust-analyzer/issues/7839. Specifically, when extracting block expressions, remove the unneeded extra braces inside the generated function. Co-authored-by: Brandon <[email protected]>
| * | | | Remove unnecessary braces for extracted block expressionBrandon2021-04-221-14/+22
| | | | |
* | | | | Merge #8621bors[bot]2021-04-222-0/+21
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8621: feat: make sure nightly regressions don't break users r=lnicola a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | feat: make sure nightly regressions don't break usersAleksey Kladov2021-04-222-0/+21
| | | | | |
* | | | | | 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]>