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

Rollup of 20 pull requests #27209

Closed
wants to merge 53 commits into from
Closed

Conversation

jawline and others added 30 commits July 4, 2015 23:12
Print error followed by expected to make the error more clear
The fix for rust-lang#26468 was made upstream and landed with the LLVM update in rust-lang#27076.

Closes rust-lang#26468
I'm pretty sure this was a typo
Make them all consistent and link up the documentation.
These provide various special readers, so point their docs to their
constructor functions in a manner consistent with everything else.
Better and more consistent links to their creators.
This should help people configure travis to test all three channels.
Beef up the struct docs, add examples for the methods.
The fix for rust-lang#26468 was made upstream and landed with the LLVM update in rust-lang#27076.

Closes rust-lang#26468
… r=alexcrichton

Mostly through adding examples.

r? @alexcrichton 

I'm going to be doing a bunch of these today, but I figured I'd keep it one PR per struct, since the last 'all the things in one PR' ended up taking a week to actually land.
…arth

This is a resubmission of my previous git failure - apologies.  Just fixing up a wording error that was discovered in E0253 after the r.

r? @Manishearth
…r=alexcrichton

Make them all consistent and link up the documentation.

r? @alexcrichton
…ichton

These provide various special readers, so point their docs to their
constructor functions in a manner consistent with everything else.

r? @alexcrichton
… r=alexcrichton

Beef up the struct docs, add examples for the methods.

r? @alexcrichton
…xcrichton

Better and more consistent links to their creators.
…r=alexcrichton

This should help people configure travis to test all three channels.
@steveklabnik
Copy link
Member Author

@bors: r+ p=1000000000000000000000000000000000000000000

@bors
Copy link
Collaborator

bors commented Jul 22, 2015

📌 Commit 9c8a0d9 has been approved by steveklabnik

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@bors
Copy link
Collaborator

bors commented Jul 22, 2015

⌛ Testing commit 9c8a0d9 with merge e7573f9...

@barosl
Copy link
Contributor

barosl commented Jul 22, 2015

What a huge priority number!

@bors
Copy link
Collaborator

bors commented Jul 22, 2015

💔 Test failed - auto-mac-64-nopt-t

///
/// ```
/// use std::fs::File;
/// use std::io::prelude::*;
Copy link
Contributor

Choose a reason for hiding this comment

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

It appears that this import was unused and caused the test failure.

@steveklabnik
Copy link
Member Author

ugh :( changing examples at the last minute.... i thought i had fixed this.

@Centril Centril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.