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

Improve trait/impl method discrepancy errors #84014

Merged
merged 4 commits into from
Apr 12, 2021

Conversation

estebank
Copy link
Contributor

@estebank estebank commented Apr 8, 2021

  • Use more accurate spans
  • Clean up some code by removing previous hack
  • Provide structured suggestions

Structured suggestions are particularly useful for cases where arbitrary self types are used, like in custom Futures, because the way to write self: Pin<&mut Self> is not necessarily self-evident when first encountered.

This is particularly useful for cases where arbitrary self types are
used, like in custom `Future`s.
@rust-highfive
Copy link
Collaborator

r? @varkor

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 8, 2021
@varkor
Copy link
Member

varkor commented Apr 11, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Apr 11, 2021

📌 Commit bb502c4 has been approved by varkor

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 11, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Apr 12, 2021
Rollup of 7 pull requests

Successful merges:

 - rust-lang#83669 (Issue 81508 fix)
 - rust-lang#84014 (Improve trait/impl method discrepancy errors)
 - rust-lang#84059 (Bump libc dependency of std to 0.2.93)
 - rust-lang#84067 (clean up example on read_to_string)
 - rust-lang#84079 (Improve test for `rustdoc::bare_urls` lint)
 - rust-lang#84094 (Remove FixedSizeArray)
 - rust-lang#84101 (rustdoc: Move crate loader to collect_intra_doc_links::early )

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit c905e9d into rust-lang:master Apr 12, 2021
@rustbot rustbot added this to the 1.53.0 milestone Apr 12, 2021
@estebank estebank deleted the cool-bears-hot-tip branch November 9, 2023 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants