Skip to content

Add long error explanation for E0625 - #87715

Merged
bors merged 3 commits into
rust-lang:masterfrom
bhgomes:long-explanation-E0625
Aug 6, 2021
Merged

Add long error explanation for E0625#87715
bors merged 3 commits into
rust-lang:masterfrom
bhgomes:long-explanation-E0625

Conversation

@bhgomes

Copy link
Copy Markdown
Contributor

For #61137.

@rust-highfive

Copy link
Copy Markdown
Contributor

Some changes occurred in diagnostic error codes

cc @GuillaumeGomez

@rust-highfive

Copy link
Copy Markdown
Contributor

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @petrochenkov (or someone else) soon.

Please see the contribution instructions for more information.

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 3, 2021
@rust-log-analyzer

This comment has been minimized.

@petrochenkov

Copy link
Copy Markdown
Contributor

tidy error: /checkout/compiler/rustc_error_codes/src/error_codes/E0625.md: too many trailing newlines (2)

@petrochenkovpetrochenkov added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 3, 2021

@GuillaumeGomezGuillaumeGomez left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please move the explanation after the failing example but otherwise looks good to me.

Comment threadcompiler/rustc_error_codes/src/error_codes/E0625.md Outdated
@petrochenkov

Copy link
Copy Markdown
Contributor

r? @GuillaumeGomez

@GuillaumeGomez

Copy link
Copy Markdown
Member

Thanks! r=me once CI pass

@GuillaumeGomez

Copy link
Copy Markdown
Member

@bors: r+ rollup squash

@bors

bors commented Aug 4, 2021

Copy link
Copy Markdown
Collaborator

📌 Commit dc5f6d2 has been approved by GuillaumeGomez

@borsbors 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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 4, 2021
JohnTitor added a commit to JohnTitor/rust that referenced this pull request Aug 6, 2021
…uillaumeGomez
Add long error explanation for E0625
For rust-lang#61137.
@JohnTitorJohnTitor mentioned this pull request Aug 6, 2021
@JohnTitorJohnTitor mentioned this pull request Aug 6, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 6, 2021
Rollup of 9 pull requests
Successful merges:
- rust-lang#87561 (thread set_name haiku implementation.)
- rust-lang#87715 (Add long error explanation for E0625)
- rust-lang#87727 (explicit_generic_args_with_impl_trait: fix min expected number of generics)
- rust-lang#87742 (Validate FFI-safety warnings on naked functions)
- rust-lang#87756 (Add back -Zno-profiler-runtime)
- rust-lang#87759 (Re-use std::sealed::Sealed in os/linux/process.)
- rust-lang#87760 (Promote `aarch64-apple-ios-sim` to Tier 2)
- rust-lang#87770 (permit drop impls with generic constants in where clauses)
- rust-lang#87780 (alloc: Use intra doc links for the reserve function)
Failed merges:
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit 772db06 into rust-lang:masterAug 6, 2021
@rustbotrustbot added this to the 1.56.0 milestone Aug 6, 2021
@bhgomes
bhgomes deleted the long-explanation-E0625 branch September 14, 2021 00:33
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: 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.

7 participants

@bhgomes@rust-highfive@rust-log-analyzer@petrochenkov@GuillaumeGomez@bors@rustbot