Skip to content

Conversation

GuillaumeGomez and others added 19 commits May 27, 2016 00:39
…expected to and was still considered 'ok')

* Fix error explanations tests/tags
Added documentation clarifying the behavior of split when used with the empty string and contiguous separators.
…uillaumeGomez

Added examples/docs to split in str.rs

Added documentation clarifying the behavior of split when used with the empty string and contiguous separators. Addresses issue [33882](rust-lang#33882). This is my first time contributing to rust, so forgive me if I'm skipping any of the contribution steps.
Fixes rust-lang#33882
…illaumeGomez

run rustfmt on librand folder
@rust-highfive
Copy link
Contributor

r? @jroesch

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

@Manishearth
Copy link
Member Author

@bors r+ p=20

@bors
Copy link
Collaborator

bors commented May 30, 2016

📌 Commit 9d2ec40 has been approved by Manishearth

@Manishearth
Copy link
Member Author

@bors force

1 similar comment
@Manishearth
Copy link
Member Author

@bors force

@bors
Copy link
Collaborator

bors commented May 30, 2016

⌛ Testing commit 9d2ec40 with merge 5da602b...

bors added a commit that referenced this pull request May 30, 2016
Rollup of 8 pull requests

- Successful merges: #33793, #33893, #33912, #33913, #33914, #33917, #33937, #33938
- Failed merges:
@bors bors mentioned this pull request May 30, 2016
@bors bors merged commit 9d2ec40 into rust-lang:master May 30, 2016
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 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.

8 participants