-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Improve E0585 help #102351
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Improve E0585 help #102351
Conversation
r? @wesleywiser (rust-highfive has picked a reviewer for you, use r? to override) |
☔ The latest upstream changes (presumably #101619) made this pull request unmergeable. Please resolve the merge conflicts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems reasonable to me!
@Rageking8 if you can rebase and resolve the merge conflict, I'd be happy to r+ |
Some changes occurred in src/tools/cargo cc @ehuss |
Oops, new to git and cargo submodules changes keep slipping in. I think I fixed it. Edit: Fix successful. |
cc @davidtwco, @compiler-errors, @JohnTitor, @estebank, @TaKO8Ki |
@wesleywiser Fixed the merge conflicts and CI failures, may I get a rereview (added a few more changes to make CI pass). Thanks. |
@bors r+ rollup |
Rollup of 8 pull requests Successful merges: - rust-lang#98368 (Make `std::os::fd` public.) - rust-lang#102085 (Code refactoring smart_resolve_report_errors) - rust-lang#102351 (Improve E0585 help) - rust-lang#102368 (Add a niche to `Duration`, unix `SystemTime`, and non-apple `Instant`) - rust-lang#102393 (Add regression test for issue 94923) - rust-lang#102399 (Account for use of index-based lifetime names in print of binder) - rust-lang#102416 (remove FIXME, improve documentation) - rust-lang#102433 (env::temp_dir: fix a typo) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
No description provided.