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

Recover from missing associated items and generic const defaults #15212

Merged
merged 1 commit into from Jul 4, 2023

Conversation

lowr
Copy link
Contributor

@lowr lowr commented Jul 4, 2023

Fixes #15129

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 4, 2023
@lowr lowr force-pushed the patch/recover-from-incomplete-assoc-ty branch 2 times, most recently from ddeb6af to abbe85e Compare July 4, 2023 13:23
@lnicola
Copy link
Member

lnicola commented Jul 4, 2023

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 4, 2023

📌 Commit abbe85e has been approved by lnicola

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Jul 4, 2023

⌛ Testing commit abbe85e with merge 3838884...

bors added a commit that referenced this pull request Jul 4, 2023
…=lnicola

Recover from missing associated item binding

Fixes #15129
@lowr
Copy link
Contributor Author

lowr commented Jul 4, 2023

Oh wait a sec, we want to do the same thing for const param default I guess.

@bors r-

@bors
Copy link
Collaborator

bors commented Jul 4, 2023

☀️ Try build successful - checks-actions
Build commit: 3838884 (383888442f02cc763c0de38e9c1e95cc23848368)

@lowr lowr force-pushed the patch/recover-from-incomplete-assoc-ty branch from abbe85e to 49b039f Compare July 4, 2023 15:27
@lowr lowr changed the title Recover from missing associated item binding Recover from missing associated items and generic const defaults Jul 4, 2023
@HKalbasi
Copy link
Member

HKalbasi commented Jul 4, 2023

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 4, 2023

📌 Commit 49b039f has been approved by HKalbasi

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Jul 4, 2023

⌛ Testing commit 49b039f with merge e95644e...

@bors
Copy link
Collaborator

bors commented Jul 4, 2023

☀️ Test successful - checks-actions
Approved by: HKalbasi
Pushing e95644e to master...

@bors bors merged commit e95644e into rust-lang:master Jul 4, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missin path in assoc type arg eagerly parses >
5 participants