Skip to content

Slightly improve code samples in E0591 #79486

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

Merged
merged 1 commit into from
Nov 28, 2020

Conversation

camelid
Copy link
Member

@camelid camelid commented Nov 28, 2020

  • Improve formatting
  • Don't hide unsafe block - it's important!

@camelid camelid added the A-error-codes Area: Explanation of an error code (--explain) label Nov 28, 2020
@rust-highfive
Copy link
Contributor

r? @lcnr

(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 Nov 28, 2020
@camelid
Copy link
Member Author

camelid commented Nov 28, 2020

What luck: I chose to change the one error code that is used in the --explain UI test 😄

* Improve formatting
* Don't hide `unsafe` block - it's important!
@lcnr
Copy link
Contributor

lcnr commented Nov 28, 2020

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Nov 28, 2020

📌 Commit 1b846bf has been approved by lcnr

@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 Nov 28, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 28, 2020
…as-schievink

Rollup of 10 pull requests

Successful merges:

 - rust-lang#78086 (Improve doc for 'as _')
 - rust-lang#78853 (rustc_parse: fix ConstBlock expr span)
 - rust-lang#79234 (Resolve typedefs in HashMap gdb/lldb pretty-printers)
 - rust-lang#79344 (Convert UNC path to local path to satisfy install script on Windows)
 - rust-lang#79383 (Fix bold code formatting in keyword docs)
 - rust-lang#79460 (Remove intermediate vectors from `add_bounds`)
 - rust-lang#79474 (Change comments on types to doc-comments)
 - rust-lang#79476 (Sync rustc_codegen_cranelift)
 - rust-lang#79478 (Expand docs on Peekable::peek_mut)
 - rust-lang#79486 (Slightly improve code samples in E0591)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 208d680 into rust-lang:master Nov 28, 2020
@rustbot rustbot added this to the 1.50.0 milestone Nov 28, 2020
@camelid camelid deleted the E0591-code-cleanup branch November 28, 2020 20:14
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A-error-codes Area: Explanation of an error code (--explain) 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