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 parameter for Interner in derive(FallibleTypeFolder) macro #779

Merged
merged 1 commit into from
Oct 13, 2022

Conversation

lowr
Copy link
Contributor

@lowr lowr commented Oct 13, 2022

In custom derive macro for FallibleTypeFolder introduced in #772, there's one occurrence of raw type parameter that is not guaranteed to exist. This patch replaces it with #interner to be interpolated at proc macro runtime like in other methods.

@jackh726
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Oct 13, 2022

📌 Commit f299265 has been approved by jackh726

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Oct 13, 2022

⌛ Testing commit f299265 with merge 8138a99...

@bors
Copy link
Contributor

bors commented Oct 13, 2022

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

@bors bors merged commit 8138a99 into rust-lang:master Oct 13, 2022
bors added a commit to rust-lang/rust-analyzer that referenced this pull request Oct 16, 2022
Bump chalk

There's a bug in current chalk that prevents us from properly supporting GATs, which is supposed to be fixed in v0.86. Note the following:
- v0.86 is only going to be released next Sunday so I'll keep this PR as draft until then.
- This doesn't compile without rust-lang/chalk#779, which I hope will be included in v0.86. I confirmed this compiles with it locally.

Two breaking changes from v0.84:
- `TypeFolder` has been split into `TypeFolder` and `FallibleTypeFolder` (rust-lang/chalk#772)
- `ProjectionTy::self_type_parameter()` has been removed (rust-lang/chalk#778)
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