Skip to content

Rollup of 13 pull requests - #27194

Closed
steveklabnik wants to merge 32 commits into
rust-lang:masterfrom
steveklabnik:rollup
Closed

Rollup of 13 pull requests#27194
steveklabnik wants to merge 32 commits into
rust-lang:masterfrom
steveklabnik:rollup

Conversation

jawlineand 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.
Closesrust-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.
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.
Closesrust-lang#26468
…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
…xcrichton
Better and more consistent links to their creators.
@steveklabnik

Copy link
Copy Markdown
ContributorAuthor

@bors: r+ p=1000

@bors

bors commented Jul 21, 2015

Copy link
Copy Markdown
Collaborator

📌 Commit da8036e has been approved by steveklabnik

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @alexcrichton

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

@bors

bors commented Jul 21, 2015

Copy link
Copy Markdown
Collaborator

⌛ Testing commit da8036e with merge 714da08...

@bors

bors commented Jul 21, 2015

Copy link
Copy Markdown
Collaborator

💔 Test failed - auto-mac-64-opt

@steveklabnik

Copy link
Copy Markdown
ContributorAuthor

@bors: retry

@apasel422

Copy link
Copy Markdown
Contributor

@steveklabnik Can you replace 5a2bcd3 with e6d8434?

@alexcrichton

Copy link
Copy Markdown
Member

@bors: r-

Unfortunately I think the test failure was legitimate.

@steveklabnik

Copy link
Copy Markdown
ContributorAuthor

closing, and I will fix up and make a new rollup soon

@CentrilCentril added the rollup A PR which is a rollup label Oct 24, 2019
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rollupA PR which is a rollup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants

@steveklabnik@bors@rust-highfive@apasel422@alexcrichton@Centril@jawline@dotdash@pfalabella@AlisdairO