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

Extend E0106, E0261 #57950

Merged
merged 3 commits into from
Jan 29, 2019
Merged

Extend E0106, E0261 #57950

merged 3 commits into from
Jan 29, 2019

Conversation

QuietMisdreavus
Copy link
Member

This is a reopening of #57310 with review comments addressed because the original author has since deleted their fork.

From the author (@purple-ice):

Added an example that points out hardly obvious mistake one could make when writing impl for a new type.

r? @rust-lang/docs

purple-ice and others added 3 commits January 3, 2019 20:15
Added an example that points out hardly obvious mistake one could make when writing impl for a new type.
Replaced impl block dock with suggested one and made sure that blocks compile.
@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 28, 2019
@Centril
Copy link
Contributor

Centril commented Jan 28, 2019

r? @estebank

@estebank
Copy link
Contributor

r=me after travis is happy

@Centril
Copy link
Contributor

Centril commented Jan 28, 2019

@bors r=estebank rollup

@bors
Copy link
Contributor

bors commented Jan 28, 2019

📌 Commit a2b75ed has been approved by estebank

@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 28, 2019
Centril added a commit to Centril/rust that referenced this pull request Jan 28, 2019
…r=estebank

Extend E0106, E0261

This is a reopening of rust-lang#57310 with review comments addressed because the original author has since deleted their fork.

From the author (@purple-ice):

> Added an example that points out hardly obvious mistake one could make when writing impl for a new type.

r? @rust-lang/docs
bors added a commit that referenced this pull request Jan 29, 2019
Rollup of 7 pull requests

Successful merges:

 - #57045 (Kill remaining uses of mem::uninitialized in libcore, liballoc)
 - #57674 (Avoid erase_regions_ty queries if there are no regions to erase)
 - #57833 (Print a slightly clearer message when failing to launch a thread)
 - #57859 (Fix invalid background color)
 - #57904 (add typo suggestion to unknown attribute error)
 - #57915 (Pretty print `$crate` as `crate` or `crate_name` in more cases)
 - #57950 (Extend E0106, E0261)

Failed merges:

r? @ghost
@bors bors merged commit a2b75ed into rust-lang:master Jan 29, 2019
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.

6 participants