Skip to content
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

align rustc and chalk folder type #31

Open
2 tasks done
nikomatsakis opened this issue May 18, 2021 · 11 comments
Open
2 tasks done

align rustc and chalk folder type #31

nikomatsakis opened this issue May 18, 2021 · 11 comments

Comments

@nikomatsakis
Copy link
Contributor

nikomatsakis commented May 18, 2021

@nikomatsakis

This comment has been minimized.

@rustbot

This comment has been minimized.

@nikomatsakis
Copy link
Contributor Author

@rustbot assign @LeSeulArtichaut

@LeSeulArtichaut

This comment has been minimized.

@nikomatsakis
Copy link
Contributor Author

(cc @rust-lang/release -- any tips on how to make the triagebot work :)

@Mark-Simulacrum
Copy link
Member

(Triagebot should work now, fixed the permissions).

@eggyal
Copy link

eggyal commented Nov 25, 2021

I understand from rust-lang/rust#85485 (comment) that @LeSeulArtichaut isn't able to work on these at present. Would it be okay if I have a crack at moving them forward? I've already opened rust-lang/rust#91230 to pick up where rust-lang/rust#85469 left off.

@rustbot claim

@rustbot rustbot assigned eggyal and unassigned LeSeulArtichaut Nov 25, 2021
@eggyal
Copy link

eggyal commented Nov 28, 2021

With rust-lang/rust#91230 merged, the first of these actions can be marked complete.

@eggyal eggyal removed their assignment Jun 6, 2022
@eggyal
Copy link

eggyal commented Feb 5, 2023

The second action here had me confused for a while, until I spotted that TypeFamily was renamed Interner in rust-lang/chalk#329.

@eggyal
Copy link

eggyal commented Feb 14, 2023

With rust-lang/rust#107924 merged, rustc's folder is also now generic over the Interner (formerly Type Family).

And since chalk is no longer to be integrated with rustc, I guess further alignment of the folder types is not required?

@jackh726
Copy link
Member

Well, how are they still different? Might give some clues on if the rustc folder traits should be modified for some future solver state

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

No branches or pull requests

6 participants