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

show how iterating over RangeTo and RangeToInclusive fails #35771

Merged
merged 1 commit into from
Aug 30, 2016

Conversation

matthew-piziak
Copy link
Contributor

Feedback on PR #35701 seems to be positive, so this does the same thing for RangeTo and RangeToInclusive.

@rust-highfive
Copy link
Collaborator

r? @aturon

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

@matthew-piziak
Copy link
Contributor Author

r? @steveklabnik

@rust-highfive rust-highfive assigned steveklabnik and unassigned aturon Aug 17, 2016
@steveklabnik
Copy link
Member

Could we make the same style changes here as the other one, where the examples are split up, please?

@matthew-piziak
Copy link
Contributor Author

matthew-piziak commented Aug 20, 2016

↓ done

Feedback on PR rust-lang#35701 seems to be positive, so this does the same thing for `RangeTo` and `RangeToInclusive`.
@frewsxcv
Copy link
Member

Looks good! 👍

@steveklabnik
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Aug 25, 2016

📌 Commit 2659198 has been approved by steveklabnik

@bors
Copy link
Contributor

bors commented Aug 29, 2016

⌛ Testing commit 2659198 with merge f4596a1...

@bors
Copy link
Contributor

bors commented Aug 29, 2016

💔 Test failed - auto-mac-64-opt-rustbuild

@arielb1
Copy link
Contributor

arielb1 commented Aug 29, 2016

@bors retry

bors added a commit that referenced this pull request Aug 29, 2016
bors added a commit that referenced this pull request Aug 30, 2016
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Aug 30, 2016
…ple-error, r=steveklabnik

show how iterating over `RangeTo` and `RangeToInclusive` fails

Feedback on PR rust-lang#35701 seems to be positive, so this does the same thing for `RangeTo` and `RangeToInclusive`.
bors added a commit that referenced this pull request Aug 30, 2016
@bors bors merged commit 2659198 into rust-lang:master Aug 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants