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

fix: consider ADT's generic parameters #781

Merged
merged 1 commit into from
Oct 21, 2022

Conversation

lowr
Copy link
Contributor

@lowr lowr commented Oct 21, 2022

Fixes #762

chalk-integration has been overlooking ADTs' generic parameters when it generates auto trait impls for them. This function is only used to print the lowered program in cli so it doesn't affect other crates.

@jackh726
Copy link
Member

@bors r+

@bors
Copy link
Collaborator

bors commented Oct 21, 2022

📌 Commit 64632c0 has been approved by jackh726

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Oct 21, 2022

⌛ Testing commit 64632c0 with merge cadaba9...

@bors
Copy link
Collaborator

bors commented Oct 21, 2022

☀️ Test successful - checks-actions
Approved by: jackh726
Pushing cadaba9 to master...

@bors bors merged commit cadaba9 into rust-lang:master Oct 21, 2022
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.

Lowered command panics with auto trait
3 participants