-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Add an extra note to late_bound_lifetime_arguments
error/lint
#43343
Conversation
r? @pnkfelix (rust_highfive has picked a reviewer for you, use r? to override) |
r? @estebank |
@bors r+ |
📌 Commit 18d9701 has been approved by |
⌛ Testing commit 18d9701 with merge 53cc5b773521da8bea78420b8eaffb4501f2637d... |
💔 Test failed - status-travis |
Looks like some infra issue, not sure spurious or not.
|
⌛ Testing commit 18d9701 with merge 5be21773a9ab31d3c372f60c237e7b3707ee9104... |
Got the same error in #43319 |
💔 Test failed - status-travis |
@bors retry (a current retried build appears to have got past the issue, so the cached dockerfile should be used from now on which will hopefully resolve the issue) |
Add an extra note to `late_bound_lifetime_arguments` error/lint Fixes rust-lang#42868 (comment)
⌛ Testing commit 18d9701 with merge 8fed6814e1668cf8d4cb3c93ed54206b7c98734b... |
💔 Test failed - status-appveyor |
That's an odd error:
|
I've seen it before and it was transient |
@petrochenkov out of curiosity, do you know if there's an issue filed for that spurious failure? |
Thanks! I've filed an issue for that. |
Add an extra note to `late_bound_lifetime_arguments` error/lint Fixes #42868 (comment)
☀️ Test successful - status-appveyor, status-travis |
Fixes #42868 (comment)