Skip to content

Migrate variance to the next solver and remove lint allows from its stuff#20867

Merged
ShoyuVanilla merged 2 commits into
rust-lang:masterfrom
ChayimFriedman2:to-ns-almost-final
Oct 20, 2025
Merged

Migrate variance to the next solver and remove lint allows from its stuff#20867
ShoyuVanilla merged 2 commits into
rust-lang:masterfrom
ChayimFriedman2:to-ns-almost-final

Conversation

@ChayimFriedman2

Copy link
Copy Markdown
Contributor

Reviewing commit-by-commit is recommended. The second commit could use a more quick review, it was mostly following the compiler.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 19, 2025

@ShoyuVanilla ShoyuVanilla left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, though I just realized how many unused params - mostly unimplemented/unnecessary trait impls from rustc_type_ir, which are fine - we have 😅

@ShoyuVanilla ShoyuVanilla added this pull request to the merge queue Oct 20, 2025
Merged via the queue into rust-lang:master with commit 9c494da Oct 20, 2025
15 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 20, 2025
@ChayimFriedman2 ChayimFriedman2 deleted the to-ns-almost-final branch October 20, 2025 08:20
@lnicola

lnicola commented Oct 24, 2025

Copy link
Copy Markdown
Member

changelog fixup #20329

mendelsshop pushed a commit to mendelsshop/rust-analyzer that referenced this pull request Oct 27, 2025
…final

Migrate variance to the next solver and remove lint allows from its stuff
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants