-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Spell out Self
in async function return
#72260
Conversation
Some changes occurred in diagnostic error codes |
8a64343
to
755ca01
Compare
This comment has been minimized.
This comment has been minimized.
0c314b9
to
142b1d3
Compare
This comment has been minimized.
This comment has been minimized.
84f6b27
to
2d9c166
Compare
Looks good, thanks! @bors r+ |
📌 Commit 2f311b0 has been approved by |
Spell out `Self` in async function return Closes rust-lang#69276 r? @tmandry
Failed in #72430 (comment)
@bors r- |
r? @RalfJung |
Sorry, I can't review this, I don't know the code. I just saw a failure during a rollup. |
@tmandry ping |
This PR was approved, but dequeued from the merging list as error-code conflict. |
@csmoe there's another PR that has taken E0758 already (and I have one open for E0759). Update and I'll r+ |
@estebank moved to E0760. |
@bors r+ |
📌 Commit 9be6353 has been approved by |
Spell out `Self` in async function return Closes rust-lang#69276 r? @tmandry
Spell out `Self` in async function return Closes rust-lang#69276 r? @tmandry
Rollup of 5 pull requests Successful merges: - rust-lang#72260 (Spell out `Self` in async function return) - rust-lang#72996 (Remove unsused `NodeId` related APIs in hir map) - rust-lang#73010 (Update RELEASES.md) - rust-lang#73017 (Use assert_eq for liballoc test) - rust-lang#73019 (add test for rust-lang#72960) Failed merges: r? @ghost
Closes #69276
r? @tmandry