Add long explanation for E0547#81836
Add long explanation for E0547#81836jesusprubio wants to merge 2 commits intorust-lang:masterfrom jesusprubio:add-long-explanation-e0547
Conversation
|
Some changes occurred in diagnostic error codes |
|
r? @lcnr (rust-highfive has picked a reviewer for you, use r? to override) |
rust-highfive
added
the
S-waiting-on-review
|
Thanks! @bors: r+ rollup squash |
|
📌 Commit 3691878 has been approved by |
bors
added
S-waiting-on-bors
…laumeGomez Rollup of 5 pull requests Successful merges: - rust-lang#81526 (btree: use Option's unwrap_unchecked()) - rust-lang#81742 (Add a note about the correctness and the effect on unsafe code to the `ExactSizeIterator` docs) - rust-lang#81830 (Add long error explanation for E0542) - rust-lang#81835 (Improve long explanation for E0546) - rust-lang#81843 (Add regression test for rust-lang#29821) Failed merges: - rust-lang#81836 (Add long explanation for E0547) r? `@ghost` `@rustbot` modify labels: rollup
|
☔ The latest upstream changes (presumably #81853) made this pull request unmergeable. Please resolve the merge conflicts. |
bors
added
S-waiting-on-author
|
Generally we use rebase instead of merge to fix conflicts. However, I don't know if it's a requirement or not... |
| @@ -0,0 +1,37 @@ | |||
| The `issue` value is missing in a stability attribute. | |||
There was a problem hiding this comment.
Would this be clearer if it is flipped? I wonder if the term is called "value"?
| The `issue` value is missing in a stability attribute. | |
| A stability attribute is missing `issue`. |
There was a problem hiding this comment.
Honestly, I can't see any benefit. Anyway, it's not a big deal for me :).
@GuillaumeGomez , what do you think?
There was a problem hiding this comment.
I find the suggested one more difficult to understand.
My fault :), I'm starting a new PR and closing this one. |
|
Closing in favor of: #81925 |
6 participants
Helps with #61137