Skip to content

Subtree update of rust-analyzer #140385

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 426 commits into
base: master
Choose a base branch
from
Open

Conversation

lnicola
Copy link
Member

@lnicola lnicola commented Apr 28, 2025

r? @ghost

Veykril and others added 30 commits March 29, 2025 07:24
fix: Fix, clarify and require a value for `proc_macro_cwd` of `CrateData`
Fix debug sourceFileMap when using cppvsdbg
fix: Cleanup param name inlay hint filtering
…s-report

fix: Fix panic in progress due to splitting unicode incorrectly
We can compute the wanted information from the block's interned data.
Avoid relying on `block_def_map()` needlessly
It doesn't actually work with merge groups ...
chore: Replace some `invoke`s with `invoke_actual` in hir-ty
…er bug in MBE

Specifically, rust-lang#18744 was the PR that was supposed to fix the old bug, but it fixed it incorrectly (and didn't add a test!) The underlying reason was that we marked metavariables in expansions as joint if they were joint in the macro call, which is incorrect.

This wrong fix causes other bug, rust-lang#19497, which this PR fixes by removing the old (incorrect) fix.
This was being used by a single assist, which qualifies under the "refactor"
kind. The variant has been removed, and all usages updated accordingly.

Signed-off-by: Prajwal S N <[email protected]>
fix(ide-assists): remove `AssistKind::None`
…s-string

Fix color brackets in string context
Signed-off-by: Prajwal S N <[email protected]>
A Cargo project can now be built without any dependency metadata being fetched.

Signed-off-by: Prajwal S N <[email protected]>
Implicit field references during struct initialization were
being dropped because get_definition was returning None because
there were multiple definitions.

This adds a new helper, `get_defintions`, that supports returning
more than one definition for a given token and hooks it up.

Fixes rust-lang#19393
Veykril and others added 19 commits April 24, 2025 08:45
fix: Fix type argument mismatch incorrectly triggering on inferred trait args
Always error when failed to parse DiscoverProjectMessage
This saves 18mb on `analysis-stats .`, without regressing speed.
And adapt `intern` to the changes in the API.
internal: More `shrink_to_fit()` and upgrade dashmap and hashbrown
"to arm expression" is hard to parse, because "arm" can be a verb. Not
all Rust users may know that "arm" refers to a `match` expression
either.

Tweak the wording to make the sentence easier to parse, and clarify
that this assist refers to `match`. Use the same wording style for the
closure version too.
adds an assist that suggests removing any underscore prefixes from used
variables.
…erscore_for_used_var

feat: adds an assist to remove underscores from used variables
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 28, 2025
@rustbot
Copy link
Collaborator

rustbot commented Apr 28, 2025

rust-analyzer is developed in its own repository. If possible, consider making this change to rust-lang/rust-analyzer instead.

cc @rust-lang/rust-analyzer

@rust-log-analyzer

This comment has been minimized.

@rustbot rustbot added A-tidy Area: The tidy tool T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) labels Apr 28, 2025
@rustbot
Copy link
Collaborator

rustbot commented Apr 28, 2025

The list of allowed third-party dependencies may have been modified! You must ensure that any new dependencies have compatible licenses before merging.

cc @davidtwco, @wesleywiser

@rust-log-analyzer

This comment has been minimized.

@lnicola
Copy link
Member Author

lnicola commented Apr 28, 2025

Quick r? on the license thing, please @davidtwco, @wesleywiser?

@rustbot
Copy link
Collaborator

rustbot commented Apr 28, 2025

Failed to set assignee to on: invalid assignee

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-tidy Area: The tidy tool S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Projects
None yet
Development

Successfully merging this pull request may close these issues.