-
Notifications
You must be signed in to change notification settings - Fork 13k
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
fix error code in E0751.md #71426
fix error code in E0751.md #71426
Conversation
r? @estebank (rust_highfive has picked a reviewer for you, use r? to override) |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
The issue is not just the annotation but also that code emits E0748 actually. |
☝️ Fixing #71304 requires not only changing the test, but rather making that code emit the right error. |
Thanks. I thought it was just the annotation. I will change the code. |
18d77d5
to
baac961
Compare
@bors r+ rollup |
📌 Commit baac961 has been approved by |
…tebank fix error code in E0751.md reference: rust-lang#71304
Rollup of 6 pull requests Successful merges: - rust-lang#70845 (Make the `structural_match` error diagnostic for const generics clearer) - rust-lang#71063 (Document unsafety in core::{option, hash}) - rust-lang#71068 (Stabilize UNICODE_VERSION (feature unicode_version)) - rust-lang#71426 (fix error code in E0751.md) - rust-lang#71459 (Add leading 0x to offset in Debug fmt of Pointer) - rust-lang#71492 (Document unsafety in core::{panicking, alloc::layout, hint, iter::adapters::zip}) Failed merges: r? @ghost
discussed at this weeks T-compiler meeting. at the time, we declined to accept for backport. but further discussion with @estebank led me to rethink that outcome. (We knew it was very low risk; we just didn't understand what the supposed gain was until estebank pointed out that the publicly posted docs would carry the incorrect number, which is not hidden by any part of our nightly/stability firewall.) So I am going to unilaterally beta-accept based on that outcome. This will also serve as an example of a new policy going forward, where very low-risk beta-nominated issues can be unilaterally beta-accepted by a T-compiler lead (and we will just announce such beta-acceptances at the meeting, which I am now doing by marking this as I-nominated.) |
…ulacrum [beta] backports This backport rollup includes the following: * [beta] Update cargo rust-lang#71967 * Backport 1.43.1 release notes to master rust-lang#71914 * Update the `cc` crate rust-lang#71882 * resolve: Relax fresh binding disambiguation slightly to fix regression rust-lang#71846 * Do not try to find binop method on RHS `TyErr` rust-lang#71810 * Quick and dirty fix of the unused_braces lint rust-lang#71517 * Update stdarch submodule rust-lang#71495 * normalize field projection ty to fix broken MIR issue rust-lang#71488 * Update openssl-src to 1.1.1g rust-lang#71430 * fix error code in E0751.md rust-lang#71426 * Don't fuse Chain in its second iterator rust-lang#71404 * Remove some `Vec` allocations to improve performance rust-lang#71268 * [CI] Use the latest Python available on Windows rust-lang#71995 It also switches the bootstrap compiler to 1.43.1.
reference: #71304