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 8 pull requests #42233

Closed
wants to merge 19 commits into from
Closed

Rollup of 8 pull requests #42233

wants to merge 19 commits into from

Conversation

@frewsxcv
Copy link
Member Author

@bors r+ p=10

@bors
Copy link
Contributor

bors commented May 26, 2017

📌 Commit 8ef72db has been approved by frewsxcv

@bors
Copy link
Contributor

bors commented May 26, 2017

⌛ Testing commit 8ef72db with merge eb3bfb4...

@bors
Copy link
Contributor

bors commented May 26, 2017

💔 Test failed - status-appveyor

@est31
Copy link
Member

est31 commented May 26, 2017

Most likely cause for the failure: #42225. See #42225 (comment)

citizen428 and others added 3 commits May 26, 2017 15:49
…excrichton

Give step_trait a distinct tracking issue from step_by

iterator_step_by has decoupled their futures, so the tracking issue should split.

Old issue: rust-lang#27741
New issue: rust-lang#42168

r? @alexcrichton (another follow-up to closed PR rust-lang#42110 (comment))
…r=Mark-Simulacrum

Remove superfluous `;;` sequences

Ran across a doubled `;;` in the docstring for `str::split`. Grep found a few more. :)
…petrochenkov

Fix 'associate type' typo

I came across an error message mentioning an 'associate type'.

Since this is the only instance of this term in rustc (it's 'associated type' everywhere else), I think this might be a typo.
…slicing, r=GuillaumeGomez

Update documentation for indexing/slicing methods

See rust-lang#39911

r? @steveklabnik
@frewsxcv frewsxcv closed this May 26, 2017
@frewsxcv frewsxcv deleted the rollup branch May 26, 2017 14:20
@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.

10 participants