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

Clarify documentation for group_by #375

Merged
merged 1 commit into from
Dec 3, 2019

Conversation

phimuemue
Copy link
Member

This refers to #374 (comment).

I tried to make the docs a bit clearer by collecting into a vector, showing the groups explicitly.

@gin-ahirsch gin-ahirsch mentioned this pull request Nov 6, 2019
@jswrenn
Copy link
Member

jswrenn commented Dec 3, 2019

Thanks for the contribution!

bors r+

@jswrenn jswrenn added this to the next milestone Dec 3, 2019
bors bot added a commit that referenced this pull request Dec 3, 2019
375: Clarify documentation for group_by r=jswrenn a=phimuemue

This refers to #374 (comment).

I tried to make the docs a bit clearer by `collect`ing into a vector, showing the groups explicitly.

Co-authored-by: philipp <[email protected]>
@jswrenn
Copy link
Member

jswrenn commented Dec 3, 2019

bors retry

bors bot added a commit that referenced this pull request Dec 3, 2019
375: Clarify documentation for group_by r=jswrenn a=phimuemue

This refers to #374 (comment).

I tried to make the docs a bit clearer by `collect`ing into a vector, showing the groups explicitly.

Co-authored-by: philipp <[email protected]>
@bors
Copy link
Contributor

bors bot commented Dec 3, 2019

Build succeeded

@bors bors bot merged commit 97c6265 into rust-itertools:master Dec 3, 2019
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.

2 participants