Skip to content

[docs] Minor wording changes to drain_filter docs - #48438

Merged
bors merged 1 commit into
rust-lang:masterfrom
mbrubeck:docs
Feb 24, 2018
Merged

[docs] Minor wording changes to drain_filter docs#48438
bors merged 1 commit into
rust-lang:masterfrom
mbrubeck:docs

Conversation

@mbrubeck

Copy link
Copy Markdown
Contributor

The docs currently say, "If the closure returns false, it will try again, and call the closure on the next element." But this happens regardless of whether the closure returns true or false.

The docs currently say, "If the closure returns false, it will try
again, and call the closure on the next element." But this happens
regardless of whether the closure returns true or false.
@rust-highfive

Copy link
Copy Markdown
Contributor

r? @TimNN

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

@rust-highfiverust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 22, 2018
@TimNN

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@bors

bors commented Feb 22, 2018

Copy link
Copy Markdown
Collaborator

📌 Commit 311fbc9 has been approved by TimNN

@borsbors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 22, 2018
Manishearth added a commit to Manishearth/rust that referenced this pull request Feb 23, 2018
[docs] Minor wording changes to drain_filter docs
The docs currently say, "If the closure returns false, it will try again, and call the closure on the next element." But this happens regardless of whether the closure returns true or false.
bors added a commit that referenced this pull request Feb 23, 2018
Rollup of 12 pull requests
- Successful merges: #47933, #48072, #48083, #48123, #48157, #48219, #48221, #48245, #48429, #48436, #48438, #48472
- Failed merges:
bors added a commit that referenced this pull request Feb 24, 2018
Rollup of 12 pull requests
- Successful merges: #47933, #48072, #48083, #48123, #48157, #48219, #48221, #48245, #48429, #48436, #48438, #48472
- Failed merges:
@bors
bors merged commit 311fbc9 into rust-lang:masterFeb 24, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@mbrubeck@rust-highfive@TimNN@bors