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 18 pull requests #27199

Closed
wants to merge 47 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
Conflicts:
	src/librustc/middle/ty.rs
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.
Mostly through adding examples.
Beef up the struct docs, add examples for the methods.
Print the error message and then what is expected by the repeat count so the output makes more sense when there is an error in the const expression
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=1000

@bors
Copy link
Contributor

bors commented Jul 21, 2015

📌 Commit 2b1472e has been approved by steveklabnik

@rust-highfive
Copy link
Collaborator

r? @huonw

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

@steveklabnik
Copy link
Member Author

re-rolling up with @apasel422 's fix

@bors
Copy link
Contributor

bors commented Jul 21, 2015

⌛ Testing commit 2b1472e with merge 8957945...

@bors
Copy link
Contributor

bors commented Jul 21, 2015

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

@apasel422
Copy link
Contributor

686d326 didn't update the compile-fail test for #21174. I'll submit a PR in a sec.

@apasel422
Copy link
Contributor

Done in #27202.

@steveklabnik
Copy link
Member Author

whoops, making a new one then

@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.