aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge #9265bors[bot]2021-06-141-41/+40
|\ | | | | | | | | | | | | | | | | 9265: internal: prepare to move rename to base_db r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * internal: prepare to move rename to base_dbAleksey Kladov2021-06-141-41/+40
|/ | | | | | | | | | It's better to handle magical cases upper in the stack, because it allows for better re-use of the general implementation below. So, we pull the `self` case up here. The end goal is to put `Definition::rename` to the `ide_db`, because it's a generally re-usable functionality useful for different ide features, alongside with the search which is already there.
* Merge #9263bors[bot]2021-06-141-41/+119
|\ | | | | | | | | | | | | | | | | 9263: fix: don't use display-related functionality where semantics matters r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * fix: don't use display-related functionality where semantics mattersAleksey Kladov2021-06-141-41/+119
| | | | | | | | | | | | | | NavigationTarget is strictly a UI-level thing -- it describes where the cursor should be placed when the user presses goto definition. It doesn't make any semantic guaratees about rage and focus range and, as such, is not suitable for driving renames.
* | Merge #9262bors[bot]2021-06-141-1/+15
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 9262: fix: Don't keep a trailing `self` token in import paths after `unmerge_use` r=Veykril a=Veykril Fixes #9213 bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | Don't keep a trailing self token in import paths after unmerge_useLukas Wirth2021-06-141-1/+15
|/ /
* | Merge #8951bors[bot]2021-06-144-47/+38
|\ \ | |/ |/| | | | | | | | | | | 8951: internal: migrate to vscode.FileSystem API r=matklad a=wxb1ank I encountered an error where `bootstrap()` attempts to create a directory with the path `C:\C:\...`. I couldn't find this reported anywhere else. Using the `vscode.FileSystem` API instead of the `fs` one works here. I assume the latter automatically prepends `C:\` to paths whereas the former does not. I don't know if this suggests `vscode.FileSystem` should be used in more places for consistency. Co-authored-by: wxb1ank <[email protected]>
| * Lint toolchain.tswxb1ank2021-06-021-1/+1
| |
| * migrate from `fs` to `vscode.FileSystem` APIwxb1ank2021-06-023-44/+35
| |
| * internal: use vscode.FileSystem API in main.tswxb1ank2021-06-022-5/+5
| |
* | Merge #9261bors[bot]2021-06-141-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | 9261: minor: dev/architecture.md: Fix typo in env-var quoting r=lnicola a=lf- Co-authored-by: Jade <[email protected]>
| * | Fix typo in env-var quotingJade2021-06-141-1/+1
|/ /
* | Merge #9260bors[bot]2021-06-1432-37/+37
|\ \ | | | | | | | | | | | | | | | | | | | | | 9260: tree-wide: make rustdoc links spiky so they are clickable r=matklad a=lf- Rustdoc was complaining about these while I was running with --document-private-items and I figure they should be fixed. Co-authored-by: Jade <[email protected]>
| * | tree-wide: make rustdoc links spiky so they are clickableJade2021-06-1432-37/+37
| | |
* | | Merge #9257bors[bot]2021-06-134-79/+51
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9257: internal: diagnostic code is mandatory r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | internal: diagnostic code is mandatoryAleksey Kladov2021-06-133-29/+17
| | | |
| * | | minor: dead codeAleksey Kladov2021-06-132-50/+34
| | | |
* | | | Merge #9256bors[bot]2021-06-136-1157/+965
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9256: internal: kill diagnostic sink r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | internal: kill diagnostic sinkAleksey Kladov2021-06-134-170/+19
| | | |
| * | | internal: use cov-mark rather than bailing out diagnosticAleksey Kladov2021-06-135-108/+62
| | | |
| * | | internal: refactor missing match arms diagnosticsAleksey Kladov2021-06-134-930/+935
| | | |
* | | | Merge #9255bors[bot]2021-06-138-794/+713
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9255: internal: remove DiagnosticWithFix infra r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | internal: remove DiagnosticWithFix infraAleksey Kladov2021-06-134-585/+534
| | | |
| * | | internal: refactor incorrect case diagnosticsAleksey Kladov2021-06-136-79/+49
|/ / /
* | | Merge #9253bors[bot]2021-06-1312-415/+378
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9253: internal: refactor missing or or some diagnostic r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | minorAleksey Kladov2021-06-132-38/+38
| | | |
| * | | internal: refactor find_map diagnosticAleksey Kladov2021-06-136-193/+192
| | | |
| * | | minorAleksey Kladov2021-06-133-44/+34
| | | |
| * | | internal: refactor missing or or some diagnosticAleksey Kladov2021-06-135-58/+44
| | | |
| * | | internal: refactor remove this semicolon diagnosticsAleksey Kladov2021-06-138-83/+71
| | | |
* | | | Merge #9252bors[bot]2021-06-139-762/+752
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9252: internal: refactor mismatched args count diagnostic r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | internal: refactor mismatched args count diagnosticAleksey Kladov2021-06-134-280/+279
| | | |
| * | | internal: refactor missing unsafe diagnosticAleksey Kladov2021-06-134-107/+106
| | | |
| * | | internal: refactor BreakOutsideOfLoop diagnosticAleksey Kladov2021-06-134-32/+36
| | | |
| * | | internal: refactor NoSuchField diagnosticAleksey Kladov2021-06-135-314/+300
| | | |
| * | | internal: refactor unimplemented builtin macro diagnosticAleksey Kladov2021-06-134-31/+33
| | | |
* | | | Merge #9251bors[bot]2021-06-136-35/+18
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9251: minor: simplify r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | minor: simplifyAleksey Kladov2021-06-136-35/+18
| | | |
* | | | Merge #9250bors[bot]2021-06-136-50/+84
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9250: internal: check diagnostics in all files and not just the first one r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | internal: check diagnostics in all files and not just the first oneAleksey Kladov2021-06-136-50/+84
|/ / /
* | | Merge #9249bors[bot]2021-06-1311-562/+398
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9249: internal: remove def-level diagnostics tests r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | internal: remove def-level diagnostics testsAleksey Kladov2021-06-133-171/+21
| | | |
| * | | internal: refactor macro errorAleksey Kladov2021-06-137-198/+178
| | | |
| * | | internal: refactor unresolved proc macro diagnosticAleksey Kladov2021-06-135-61/+59
| | | |
| * | | internal: refactor inactive code diagnosticsAleksey Kladov2021-06-136-133/+141
| | | |
* | | | Merge #9248bors[bot]2021-06-137-186/+220
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9248: internal: refactor unresolved macro call diagnostic r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | internal: refactor unresolved macro call diagnosticAleksey Kladov2021-06-135-95/+94
| | | |
| * | | internal: refactor unresolved import diagnosticAleksey Kladov2021-06-136-92/+127
| | | |
* | | | Merge #9247bors[bot]2021-06-135-54/+60
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9247: internal: refactor unresolved extern crate diagnostic r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | internal: refactor unresolved extern crate diagnosticAleksey Kladov2021-06-135-54/+60
| | | |