Skip to content

Conversation

@jackh726
Copy link
Member

@jackh726 jackh726 commented Aug 2, 2025

In the next-solver work for rust-analyzer, mapping consts chalk -> next-solver -> chalk loses the ty and is replaced with TyKind::Error. Let's just allow TyKind::Error in relations

Copy link
Member

@compiler-errors compiler-errors left a comment

Choose a reason for hiding this comment

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

I've got one nit, thoughts?

Copy link
Contributor

@davidbarsky davidbarsky left a comment

Choose a reason for hiding this comment

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

(approving, unsure if i can land the changes...)

edit: no, i can't :D

@compiler-errors compiler-errors merged commit 4fb0a98 into rust-lang:master Aug 4, 2025
5 checks passed
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.

3 participants