Skip to content
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

Clean up err codes #68176

Merged
merged 2 commits into from
Jan 14, 2020
Merged

Clean up err codes #68176

merged 2 commits into from
Jan 14, 2020

Conversation

GuillaumeGomez
Copy link
Member

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 13, 2020
@@ -10,8 +11,9 @@ type Foo = Trait; // error: the value of the associated type `Bar` (from
// the trait `Trait`) must be specified
```

Please verify you specified all associated types of the trait and that you
used the right trait. Example:
Trait objects need to have all associated types specified. Please verify you

Choose a reason for hiding this comment

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

Suggested change
Trait objects need to have all associated types specified. Please verify you
Trait objects need to have all associated types specified. Please verify that

Please verify you specified all associated types of the trait and that you
used the right trait. Example:
Trait objects need to have all associated types specified. Please verify you
specified all associated types of the trait and that you used the right trait.

Choose a reason for hiding this comment

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

Suggested change
specified all associated types of the trait and that you used the right trait.
all associated types of the trait were specified and the correct trait was used.

@@ -1,5 +1,6 @@
Trait objects need to have all associated types specified. Erroneous code
example:
A trait object's associated type wasn't specified.

Choose a reason for hiding this comment

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

Suggested change
A trait object's associated type wasn't specified.
An associated type wasn't specified for a trait object

@GuillaumeGomez
Copy link
Member Author

Updated.

@Dylan-DPC-zz
Copy link

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Jan 13, 2020

📌 Commit 3ec0a84 has been approved by Dylan-DPC

@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 Jan 13, 2020
JohnTitor added a commit to JohnTitor/rust that referenced this pull request Jan 14, 2020
bors added a commit that referenced this pull request Jan 14, 2020
Rollup of 10 pull requests

Successful merges:

 - #67854 (Use `report_in_external_macro` for internal lints)
 - #67989 (rustdoc: Don't allow `#![feature(...)]` on stable or beta)
 - #68036 (libterm: parse extended terminfo format)
 - #68127 (Clarify the relationship between `extended` and `tools` in `config.toml`)
 - #68143 (Forbid elided lifetimes within const generic parameter types)
 - #68150 (Document behavior of set_nonblocking on UnixListener)
 - #68166 (rustdoc: HTML escape arrows on help popup)
 - #68176 (Clean up err codes)
 - #68179 (Remove unneeded scope)
 - #68188 (Tweak assertion note in format check)

Failed merges:

r? @ghost
@bors bors merged commit 3ec0a84 into rust-lang:master Jan 14, 2020
@GuillaumeGomez GuillaumeGomez deleted the clean-up-err-codes branch January 14, 2020 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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