]>
git.lizzy.rs Git - rust.git/log
Lukas Wirth [Fri, 3 Dec 2021 23:16:47 +0000 (00:16 +0100)]
minor: Shorten spans of mismatched_arg_count diag
bors[bot] [Wed, 1 Dec 2021 12:36:17 +0000 (12:36 +0000)]
Merge #10899
10899: ide: hack to make self not unresolved reference in async trait wrapped impl's r=Veykril a=jhgg
fixes #10708
this is a bit hacky, but it "works".
i'm not sure how to even write a test for this though, but i've confirmed it works via manual testing...
Co-authored-by: Jake Heinz <jh@discordapp.com>
Jake Heinz [Wed, 1 Dec 2021 12:26:27 +0000 (12:26 +0000)]
add a fixme:
Jake Heinz [Wed, 1 Dec 2021 12:25:08 +0000 (12:25 +0000)]
simplify
Jake Heinz [Wed, 1 Dec 2021 12:16:43 +0000 (12:16 +0000)]
ide: hack to make self not unresolved reference in async trait wrapped impl's
bors[bot] [Wed, 1 Dec 2021 11:28:17 +0000 (11:28 +0000)]
Merge #10897
10897: internal: Try to fix Code tests r=lnicola a=lnicola
It looks like `--noEmit` broke our unit tests:
![image](https://user-images.githubusercontent.com/308347/
144217192 -
a8133a6b -880f-4852-846b-
b45527485336 .png)
r? `@ChayimFriedman2`
I suppose we might still:
- compile stuff twice
- not detect failing tests on CI (since we didn't notice this)
Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>
Laurențiu Nicola [Wed, 1 Dec 2021 11:09:43 +0000 (13:09 +0200)]
Bump to Node 14
Laurențiu Nicola [Wed, 1 Dec 2021 11:07:12 +0000 (13:07 +0200)]
Use xvfb-run directly
Laurențiu Nicola [Wed, 1 Dec 2021 10:52:24 +0000 (12:52 +0200)]
Bump xvfb-action to fix Code tests
bors[bot] [Wed, 1 Dec 2021 10:28:49 +0000 (10:28 +0000)]
Merge #10896
10896: hir: resolve assoc trait type in path r=jhgg a=jhgg
fixes #9802
- [ ] write tests, maybe, if this is even a good fix...
Co-authored-by: Jake Heinz <jh@discordapp.com>
Jake Heinz [Wed, 1 Dec 2021 10:28:18 +0000 (10:28 +0000)]
add test
Laurențiu Nicola [Wed, 1 Dec 2021 10:22:34 +0000 (12:22 +0200)]
Try to fix Code tests
Jake Heinz [Wed, 1 Dec 2021 09:23:42 +0000 (09:23 +0000)]
simplify??
Jake Heinz [Wed, 1 Dec 2021 08:44:30 +0000 (08:44 +0000)]
hir: resolve assoc trait type
bors[bot] [Tue, 30 Nov 2021 14:07:39 +0000 (14:07 +0000)]
Merge #10872
10872: ide_db: build symbol index from crate def map r=Veykril a=jhgg
fixes #4842, #10764
Is this looking correct? :eyes:
- [x] build the symbol index based upon the CrateDefMap for the given crate in `crate_symbols`
- [x] make it multi threaded again, and figure out how to cache each moduleid's symbol index in salsa.
- [x] NavigationTarget for names in macros is wrong, need to figure out how to compute a text range in the original file id?
- [x] cleanup some duped code
- [x] collect macros from `ItemScope.declared_macros()` into symbol index.
- [x] store declared macros in `ItemScope` so we can figure out where macros were defined for the index.
- [x] do something about `SymbolIndex::for_files` - ideally it should use the new module symbol index stuff.
- [x] delete `source_file_to_file_symbols` & co...
- [x] figure out what to do about `library_symbols`
- [x] maybe... speed up the new `library_symbols` - the new impl is probably much slower, and definitely much less parallel. **deciding to do nothing here, we can optimize later if necerssary.**
- [x] fix failing test: `navigation_target::tests::test_nav_for_symbol` - notably the crate def map doesn't seem to find declarations inside function.
- [x] now a bunch of other tests are failing around auto_import & qualify_path handlers. :(
- [x] need to assoc items in traits and impls
Co-authored-by: Jake Heinz <jh@discordapp.com>
Jake Heinz [Tue, 30 Nov 2021 08:24:07 +0000 (08:24 +0000)]
simpler way of grabbing module / trait name
Jake Heinz [Tue, 30 Nov 2021 04:50:09 +0000 (04:50 +0000)]
remove one need for semantics
Jake Heinz [Tue, 30 Nov 2021 03:21:17 +0000 (03:21 +0000)]
consts with inners??
Jake Heinz [Tue, 30 Nov 2021 03:09:38 +0000 (03:09 +0000)]
pr feedbacks
bors[bot] [Mon, 29 Nov 2021 22:03:04 +0000 (22:03 +0000)]
Merge #10887
10887: internal: Update bug report template to include version info r=Veykril a=jonas-schievink
Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>
Jonas Schievink [Mon, 29 Nov 2021 21:35:16 +0000 (22:35 +0100)]
Update bug report template to include version info
bors[bot] [Mon, 29 Nov 2021 18:02:41 +0000 (18:02 +0000)]
Merge #10871
10871: Respect `http.proxyStrictSSL` r=lnicola a=lnicola
Closes #10866
Currently untested.
Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>
bors[bot] [Mon, 29 Nov 2021 11:13:39 +0000 (11:13 +0000)]
Merge #10841
10841: Emit moniker in lsif r=Veykril a=HKalbasi
fix #10559
Co-authored-by: hkalbasi <hamidrezakalbasi@protonmail.com>
Co-authored-by: HKalbasi <45197576+HKalbasi@users.noreply.github.com>
Jake Heinz [Mon, 29 Nov 2021 10:43:51 +0000 (10:43 +0000)]
test the one thing this pr was trying to accomplish lol...
Jake Heinz [Mon, 29 Nov 2021 10:36:22 +0000 (10:36 +0000)]
add a test
Jake Heinz [Mon, 29 Nov 2021 09:52:03 +0000 (09:52 +0000)]
some more cleanups
Jake Heinz [Mon, 29 Nov 2021 09:36:00 +0000 (09:36 +0000)]
do something with library_symbols
hkalbasi [Mon, 29 Nov 2021 07:55:53 +0000 (11:25 +0330)]
fix tests
hkalbasi [Mon, 29 Nov 2021 07:40:39 +0000 (11:10 +0330)]
remove duplicate data from CrateOrigin
Jake Heinz [Mon, 29 Nov 2021 07:17:18 +0000 (07:17 +0000)]
simplify work
Jake Heinz [Mon, 29 Nov 2021 05:42:46 +0000 (05:42 +0000)]
:broom:
Jake Heinz [Mon, 29 Nov 2021 05:28:22 +0000 (05:28 +0000)]
collect macro_rules! macros into macro declarations
Jake Heinz [Mon, 29 Nov 2021 05:22:30 +0000 (05:22 +0000)]
collect macros
Jake Heinz [Mon, 29 Nov 2021 03:54:52 +0000 (03:54 +0000)]
cleanup the whole thing...
Jake Heinz [Mon, 29 Nov 2021 02:07:16 +0000 (02:07 +0000)]
traverse even more...
Jake Heinz [Mon, 29 Nov 2021 01:32:47 +0000 (01:32 +0000)]
populate container name when traversing down blocks
Jake Heinz [Mon, 29 Nov 2021 01:11:31 +0000 (01:11 +0000)]
collect blocks from unnamed consts too
Jake Heinz [Mon, 29 Nov 2021 01:09:39 +0000 (01:09 +0000)]
collect defs from body blocks
Laurențiu Nicola [Sun, 28 Nov 2021 08:54:35 +0000 (10:54 +0200)]
Refactor proxy settings
Jake Heinz [Sun, 28 Nov 2021 00:42:42 +0000 (00:42 +0000)]
cleanups
bors[bot] [Sat, 27 Nov 2021 18:14:59 +0000 (18:14 +0000)]
Merge #10876
10876: fix: Show parameter hints unconditionally for logical not expressions r=Veykril a=Veykril
Closes https://github.com/rust-analyzer/rust-analyzer/issues/8491
bors r+
Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
Lukas Wirth [Sat, 27 Nov 2021 18:12:47 +0000 (19:12 +0100)]
fix: Show parameter hints unconditionally for logical not expressions
bors[bot] [Sat, 27 Nov 2021 17:58:28 +0000 (17:58 +0000)]
Merge #10875
10875: minor: Don't discard flycheck error messages r=Veykril a=Veykril
cc #10793
bors r+
Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
Lukas Wirth [Sat, 27 Nov 2021 17:57:51 +0000 (18:57 +0100)]
Don't discard flycheck error messages
bors[bot] [Sat, 27 Nov 2021 16:12:12 +0000 (16:12 +0000)]
Merge #10874
10874: minor: correct profiling name r=matklad a=matklad
bors r+
🤖
Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>
Aleksey Kladov [Sat, 27 Nov 2021 16:11:39 +0000 (19:11 +0300)]
minor: correct profiling name
bors[bot] [Sat, 27 Nov 2021 15:18:35 +0000 (15:18 +0000)]
Merge #10873
10873: minor: if-let-else to shorter match r=matklad a=matklad
bors r+
🤖
Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>
Aleksey Kladov [Sat, 27 Nov 2021 15:18:00 +0000 (18:18 +0300)]
minor: if-let-else to shorter match
Jake Heinz [Sat, 27 Nov 2021 13:48:50 +0000 (13:48 +0000)]
make navigation target go to original file location
Jake Heinz [Sat, 27 Nov 2021 13:00:02 +0000 (13:00 +0000)]
remove file symbols
Jake Heinz [Sat, 27 Nov 2021 12:26:03 +0000 (12:26 +0000)]
container name?
Jake Heinz [Sat, 27 Nov 2021 11:55:03 +0000 (11:55 +0000)]
use Semantics::parse_or_expand in another place
Jake Heinz [Sat, 27 Nov 2021 11:49:51 +0000 (11:49 +0000)]
sema parse_or_expand
Jake Heinz [Sat, 27 Nov 2021 11:25:05 +0000 (11:25 +0000)]
wip
Laurențiu Nicola [Sat, 27 Nov 2021 05:29:44 +0000 (07:29 +0200)]
Respect http.proxyStrictSSL
bors[bot] [Sat, 27 Nov 2021 02:29:54 +0000 (02:29 +0000)]
Merge #10870
10870: ide: fix macro expansion for 'as _' r=Veykril a=jhgg
fixes: #10862
Co-authored-by: Jake Heinz <jh@discordapp.com>
Jake Heinz [Sat, 27 Nov 2021 02:22:21 +0000 (02:22 +0000)]
ide: fix expansion for 'as _'
bors[bot] [Fri, 26 Nov 2021 02:59:07 +0000 (02:59 +0000)]
Merge #10865
10865: internal: Use the right `ItemTree` when re-resolving attr r=jonas-schievink a=jonas-schievink
Followup to https://github.com/rust-analyzer/rust-analyzer/pull/10863, which caused a panic when analyzing diesel
bors r+
Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>
Jonas Schievink [Fri, 26 Nov 2021 02:57:25 +0000 (03:57 +0100)]
Use the right `ItemTree` when re-resolving attr
bors[bot] [Fri, 26 Nov 2021 00:52:02 +0000 (00:52 +0000)]
Merge #10863
10863: internal: build per-block `ItemTree`s r=Veykril a=jonas-schievink
Fixes https://github.com/rust-analyzer/rust-analyzer/issues/7717
Fixes https://github.com/rust-analyzer/rust-analyzer/issues/8911
Fixes https://github.com/rust-analyzer/rust-analyzer/issues/8614
`ItemTree`s are now flat lists of items, so they should probably be renamed at some point.
Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>
Jonas Schievink [Thu, 25 Nov 2021 23:17:20 +0000 (00:17 +0100)]
Build per-block `ItemTree`s
HKalbasi [Thu, 25 Nov 2021 16:33:43 +0000 (20:03 +0330)]
Update crates/project_model/src/workspace.rs
Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
bors[bot] [Thu, 25 Nov 2021 15:35:18 +0000 (15:35 +0000)]
Merge #10859
10859: fix: Prioritize tuple fields in highlight_related r=Veykril a=Veykril
bors r+
Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
Lukas Wirth [Thu, 25 Nov 2021 15:34:32 +0000 (16:34 +0100)]
fix: Prioritize tuple fields in highlight_related
bors[bot] [Thu, 25 Nov 2021 15:12:38 +0000 (15:12 +0000)]
Merge #10824
10824: project_model: print full cargo command if cargo metadata fails to run r=Veykril a=jhgg
well, `Command` implements a sensible `Debug` which *roughly* does what we want to do. Unfortunately it's got a bit of a quote situation,
So it'd output something like `"cargo" "metadata" "--format-version" "1" "--features" "foo,bar,baz"`.
Which although is a bit verbose with the quotes, it's at the very least, not entirely incorrect/misleading.
fixes #10797
Co-authored-by: Jake Heinz <jh@discordapp.com>
bors[bot] [Wed, 24 Nov 2021 15:33:00 +0000 (15:33 +0000)]
Merge #10847
10847: fix: derive attr path handling in `replace_derive_with_manual_impl` r=rainy-me a=rainy-me
partially fixes #10666
renaming issues mentioned in https://github.com/rust-analyzer/rust-analyzer/issues/10666#issuecomment-
955671021 doesn't seem to be fixable at this moment since it's not searchable?(not recorded?) by name.
<del>I'm not sure if it's appropriate to move `parse_comma_sep_paths` function to a method in `ast/node_ext`, maybe `ide_db::helpers` also make sense.</del> put into `ide_db::` instead.
Co-authored-by: rainy-me <github@yue.coffee>
rainy-me [Wed, 24 Nov 2021 15:21:29 +0000 (00:21 +0900)]
fix: derive path handling
bors[bot] [Wed, 24 Nov 2021 15:08:44 +0000 (15:08 +0000)]
Merge #10853
10853: fix: Remove possible multiline details in completions r=Veykril a=Veykril
Fixes https://github.com/rust-analyzer/rust-analyzer/issues/10316
Fixes https://github.com/rust-analyzer/rust-analyzer/issues/7531(this has already been fixed prior, from what I've seen)
bors r+
Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
Lukas Wirth [Wed, 24 Nov 2021 15:01:33 +0000 (16:01 +0100)]
Remove possible multiline details in completions
bors[bot] [Tue, 23 Nov 2021 21:22:27 +0000 (21:22 +0000)]
Merge #10846
10846: fix: qualify struct name in struct constructor completion r=Veykril a=andrewleverette
* Alter `add_struct_literal` method signature to take an optional module path
* Update `render_literal` method to use a qualified
Relates to issue #10771
Side note: This is my first contribution. Any feedback is welcome!
Co-authored-by: Andrew Leverette <andrewleverette@gmail.com>
bors[bot] [Tue, 23 Nov 2021 21:13:35 +0000 (21:13 +0000)]
Merge #10849
10849: fix: Disable string highlight injection for macros changing string literals r=Veykril a=Veykril
Fixes #10845
bors r+
Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
Lukas Wirth [Tue, 23 Nov 2021 21:05:52 +0000 (22:05 +0100)]
Disable string highlight injection for macros changing string literals
Andrew Leverette [Tue, 23 Nov 2021 20:41:31 +0000 (14:41 -0600)]
Add optimization for not allocating in case of no module path for struct literal
Andrew Leverette [Tue, 23 Nov 2021 16:57:29 +0000 (10:57 -0600)]
Add support for qualified struct names
bors[bot] [Tue, 23 Nov 2021 15:21:47 +0000 (15:21 +0000)]
Merge #10840
10840: fix: Omit generic defaults for types in hover messages r=jonas-schievink a=Veykril
Fixes #9198
We have ranged hovers now which query specifically for the type of an expression/pattern, so if interested in seeing the default, the user can use that functionality instead.
Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
bors[bot] [Mon, 22 Nov 2021 19:03:07 +0000 (19:03 +0000)]
Merge #10834
10834: Make lint groups work correctly with `warningsAsInfo` and `warningsAsHint` r=Veykril a=Emilgardis
this change makes it possible to specify
```json
{
"rust-analyzer.diagnostics.warningsAsInfo": ["unused"],
}
```
to make all lints in the `unused` group show as info, etc.
Co-authored-by: Emil Gardström <emil.gardstrom@gmail.com>
hkalbasi [Mon, 22 Nov 2021 17:44:46 +0000 (21:14 +0330)]
Emit moniker in lsif
Emil Gardström [Mon, 22 Nov 2021 13:14:21 +0000 (14:14 +0100)]
Make lint groups work correctly with `warningsAsInfo` and `warningsAsHint`
Lukas Wirth [Mon, 22 Nov 2021 17:27:03 +0000 (18:27 +0100)]
fix: Omit generic defaults for types in hover messages
bors[bot] [Mon, 22 Nov 2021 17:01:40 +0000 (17:01 +0000)]
Merge #10839
10839: fix: Fix mbe::Shift::new not accounting for non-ident token ids r=Veykril a=Veykril
Fixes https://github.com/rust-analyzer/rust-analyzer/issues/9371
bors r+
Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
Lukas Wirth [Mon, 22 Nov 2021 17:00:32 +0000 (18:00 +0100)]
Fix mbe::Shift::new not accounting for non-ident token ids
Lukas Wirth [Mon, 22 Nov 2021 16:58:36 +0000 (17:58 +0100)]
Document token up/down mapping
bors[bot] [Mon, 22 Nov 2021 15:59:52 +0000 (15:59 +0000)]
Merge #10838
10838: internal: Add the ability to emit token map ids in macro expansion tests r=Veykril a=Veykril
Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
Lukas Wirth [Mon, 22 Nov 2021 14:59:41 +0000 (15:59 +0100)]
Add to macro testing infra to emit token map ids
bors[bot] [Mon, 22 Nov 2021 14:29:24 +0000 (14:29 +0000)]
Merge #10809
10809: fix: don't discard formatting of `use` lines r=Veykril a=iDawer
Use mutable syntax trees in `merge_imports`, `split_imports`. This tries to resolve #9013. But I haven't much managed to simplify code of merging.
Also resolve #9361. It reuses a use tree under the cursor so that comments+indentation are preserved. Merged trees are just appended to the end.
This touches bunch of tests. I removed the sorting of use trees as it needs a proper implementation that takes into account comments and line wrapping. I think it is rustfmt's job or at least until we get a close implementation.
Co-authored-by: iDawer <ilnur.iskhakov.oss@outlook.com>
bors[bot] [Mon, 22 Nov 2021 13:47:25 +0000 (13:47 +0000)]
Merge #10835
10835: internal: pull website before generating new changelog r=jonas-schievink a=jonas-schievink
Fixes https://github.com/rust-analyzer/rust-analyzer/issues/10829
bors r+
Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>
Jonas Schievink [Mon, 22 Nov 2021 13:45:47 +0000 (14:45 +0100)]
Pull website before generating new changelog
bors[bot] [Mon, 22 Nov 2021 12:42:03 +0000 (12:42 +0000)]
Merge #10833
10833: internal: set upstream branch when promoting r=jonas-schievink a=jonas-schievink
Fixes https://github.com/rust-analyzer/rust-analyzer/issues/10830
bors r+
Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>
Jonas Schievink [Mon, 22 Nov 2021 12:39:07 +0000 (13:39 +0100)]
Set upstream branch when promoting
bors[bot] [Mon, 22 Nov 2021 12:23:58 +0000 (12:23 +0000)]
Merge #10832
10832: internal: Split parts of `ide_db::call_info` off into `ide` r=Veykril a=Veykril
`call_info` itself is just an ide feature and thus should not reside in `ide_db` itself.
bors r+
Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
Lukas Wirth [Mon, 22 Nov 2021 12:19:46 +0000 (13:19 +0100)]
Split parts of `ide_db::call_info` off into `ide`
bors[bot] [Mon, 22 Nov 2021 12:05:57 +0000 (12:05 +0000)]
Merge #10831
10831: minor: Unnest `ide::display::navigation_target` module r=Veykril a=Veykril
This module contained multiple submodules before, but those were removed some time ago so there is no point in having this one module two layers deep anymore.
bors r+
Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
Lukas Wirth [Mon, 22 Nov 2021 12:04:28 +0000 (13:04 +0100)]
Unnest ide::display::navigation_target module
bors[bot] [Mon, 22 Nov 2021 10:07:23 +0000 (10:07 +0000)]
Merge #10828
10828: doc: document absence of stability guarantees r=matklad a=matklad
bors r+
🤖
Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>
Aleksey Kladov [Mon, 22 Nov 2021 10:06:37 +0000 (13:06 +0300)]
doc: document absence of stability guarantees
bors[bot] [Sun, 21 Nov 2021 20:01:07 +0000 (20:01 +0000)]
Merge #10823
10823: fix: better `Fn` traits formatting r=jonas-schievink a=veber-alex
This makes it so formatting `Fn` traits properly handles:
1. An `Fn` trait with only a single argument -> removes the trailing comma.
2. An `Fn` trait which returns nothing (an empty tuple) -> don't show `-> ()` as the return type.
before:
![before](https://user-images.githubusercontent.com/
29788806 /
142745038 -
44ac68ce -de42-4396-a809-
ffdb883be699 .png)
after:
![after](https://user-images.githubusercontent.com/
29788806 /
142745040 -
485feaa2 -cc21-4a05-9576-
5410ea355029 .png)
Co-authored-by: Alex Veber <alexveber@gmail.com>
Jake Heinz [Sun, 21 Nov 2021 01:18:42 +0000 (01:18 +0000)]
project_model: print full cargo command if it fails to run
Alex Veber [Sun, 21 Nov 2021 00:39:22 +0000 (02:39 +0200)]
fix: better `Fn` traits formatting
bors[bot] [Sat, 20 Nov 2021 20:24:31 +0000 (20:24 +0000)]
Merge #10810
10810: feat: Add toggle to disable cache priming r=jonas-schievink a=lnicola
Even if it doesn't prevent the rest of the features from working, cache priming tends to be quite CPU-intensive and can make people think that the load times are worse than they actually are.
It's also less useful in Code and `rust-tools` because the inlay hints and semantic highlighting trigger quite a bit of computation assuming you have a file open in the editor.
Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>
bors[bot] [Sat, 20 Nov 2021 17:25:17 +0000 (17:25 +0000)]
Merge #10821
10821: fix: wrap `inline_call` and `inline_into_callers` if it inlines into the left side of a binary expression r=Veykril a=rainy-me
close #10359
Co-authored-by: rainy-me <github@yue.coffee>