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 11 pull requests #36145

Closed
wants to merge 24 commits into from
Closed

Rollup of 11 pull requests #36145

wants to merge 24 commits into from

Conversation

matthew-piziak and others added 24 commits August 26, 2016 11:54
PR rust-lang#35695 for `Range` was approved, so it seems that this side-effect-free style is preferred for Range* examples. This PR performs the same translation for `RangeFrom` and `RangeInclusive`. It also removes what looks to be an erroneously commented line for `#![feature(step_by)]`, and an unnecessary primitive-type annotation in `0u8..`.

add `fn main` wrappers to enable Rust Playground "Run" button
…tln-remaining, r=GuillaumeGomez

accumulate vector and assert for RangeFrom and RangeInclusive examples

PR rust-lang#35695 for `Range` was merged, so it seems that this side-effect-free style is preferred for Range* examples. This PR performs the same translation for `RangeFrom` and `RangeInclusive`. It also removes what looks to be an erroneously commented line for `#![feature(step_by)]`, and an unnecessary primitive-type annotation in `0u8..`.
…teveklabnik

demonstrate `RHS != Self` use cases for `Add` and `Sub`
…lexcrichton

Implement `Debug` for `std::path::{Components,Iter}`.

None
rustbook chapters/sections should be an ordered list.
doc: make TcpListener example more simple
Add test for rust-lang#14875

You can check this out in the playground https://is.gd/oVKC2T . It will fail on stable, but pass on nightly as @nagisa suggested on the issue.

Fixes rust-lang#14875
@rust-highfive
Copy link
Collaborator

r? @brson

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

@sophiajt
Copy link
Contributor Author

@bors r+ p=1

@bors
Copy link
Contributor

bors commented Aug 30, 2016

📌 Commit 251697b has been approved by jonathandturner

@bors
Copy link
Contributor

bors commented Aug 30, 2016

⌛ Testing commit 251697b with merge 75b5d2e...

@bors
Copy link
Contributor

bors commented Aug 30, 2016

💔 Test failed - auto-win-msvc-64-opt

@alexcrichton
Copy link
Member

@bors: retry

@bors
Copy link
Contributor

bors commented Aug 30, 2016

⌛ Testing commit 251697b with merge 0020804...

@bors
Copy link
Contributor

bors commented Aug 30, 2016

💔 Test failed - auto-linux-cross-opt

@alexcrichton
Copy link
Member

@bors: retry

On Tue, Aug 30, 2016 at 2:53 PM, bors notifications@github.com wrote:

💔 Test failed - auto-linux-cross-opt
https://buildbot.rust-lang.org/builders/auto-linux-cross-opt/builds/3519


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#36145 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD95FN-aqZntpwz6mXYHR2QNn7FVWoxks5qlKZEgaJpZM4Jwybv
.

@bors
Copy link
Contributor

bors commented Aug 30, 2016

⌛ Testing commit 251697b with merge 57c22f5...

@bors
Copy link
Contributor

bors commented Aug 30, 2016

💔 Test failed - auto-win-msvc-64-opt-rustbuild

@sophiajt
Copy link
Contributor Author

Closing to re-roll

@sophiajt sophiajt closed this Aug 30, 2016
@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.