Skip to content

Add long error explanation for E0584 #65493

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 2 commits into from
Oct 19, 2019

Conversation

GuillaumeGomez
Copy link
Member

Part of #61137.

r? @kinnison

@@ -295,6 +295,33 @@ named `file_that_doesnt_exist.rs` or `file_that_doesnt_exist/mod.rs` in the
same directory.
"##,

E0584: r##"
A doc comment that doesn't document anything has been.
Copy link
Contributor

Choose a reason for hiding this comment

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

...has been... what?

perhaps:

A doc comment that is not attached to anything has been encountered.

Copy link
Member Author

Choose a reason for hiding this comment

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

Woups, forgot a word! XD

@GuillaumeGomez GuillaumeGomez force-pushed the long-err-explanation-E0584 branch from ca83151 to f647c06 Compare October 17, 2019 12:19
@JohnTitor JohnTitor added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 17, 2019
Copy link
Contributor

@kinnison kinnison left a comment

Choose a reason for hiding this comment

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

@bors r=me

@GuillaumeGomez
Copy link
Member Author

@bors: r=kinnison

@bors
Copy link
Collaborator

bors commented Oct 17, 2019

📌 Commit f647c06 has been approved by kinnison

@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 Oct 17, 2019
tmandry added a commit to tmandry/rust that referenced this pull request Oct 18, 2019
…-E0584, r=kinnison

Add long error explanation for E0584

Part of rust-lang#61137.

r? @kinnison
tmandry added a commit to tmandry/rust that referenced this pull request Oct 18, 2019
…-E0584, r=kinnison

Add long error explanation for E0584

Part of rust-lang#61137.

r? @kinnison
tmandry added a commit to tmandry/rust that referenced this pull request Oct 18, 2019
…-E0584, r=kinnison

Add long error explanation for E0584

Part of rust-lang#61137.

r? @kinnison
bors added a commit that referenced this pull request Oct 19, 2019
Rollup of 19 pull requests

Successful merges:

 - #65016 (Always inline `mem::{size_of,align_of}` in debug builds)
 - #65197 (Prepare `MutVisitor`s to handle interned projections)
 - #65201 (Disable Go and OCaml bindings when building LLVM)
 - #65334 (Add long error explanation for E0575)
 - #65364 (Collect occurrences of empty blocks for mismatched braces diagnostic)
 - #65455 (Avoid unnecessary `TokenTree` to `TokenStream` conversions)
 - #65472 (Use a sharded dep node to dep node index map)
 - #65480 (Speed up `LexicalResolve::expansion()`)
 - #65493 (Add long error explanation for E0584)
 - #65496 (properly document panics in div_euclid and rem_euclid)
 - #65498 (Plugins deprecation: don’t suggest simply removing the attribute)
 - #65508 (add option to ping llvm ice-breakers to triagebot)
 - #65511 (save-analysis: Nest tables when processing impl block definitions)
 - #65513 (reorder fmt docs for more clarity)
 - #65532 (doc: make BitSet intro more short)
 - #65535 (rustc: arena-allocate the slice in `ty::GenericsPredicate`, not the whole struct.)
 - #65540 (show up some extra info when t!() fails)
 - #65549 (Fix left/right shift typo in wrapping rotate docs)
 - #65552 (Clarify diagnostics when using `~` as a unary op)

Failed merges:

 - #65390 (Add long error explanation for E0576)
 - #65434 (Add long error explanation for E0577)
 - #65471 (Add long error explanation for E0578)

r? @ghost
@bors bors merged commit f647c06 into rust-lang:master Oct 19, 2019
@GuillaumeGomez GuillaumeGomez deleted the long-err-explanation-E0584 branch October 19, 2019 08:20
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
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.

4 participants