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

Import splitWith from strawman-contrib #6

Closed
wants to merge 1 commit into from

Conversation

markus1189
Copy link

@markus1189
Copy link
Author

@julienrf I started to port the splitWith over from the strawman module.

I am currently stuck on how to get a builder for the nested collection using the new mechanism. Can you give me some pointers there? I left a TODO in the code

@markus1189
Copy link
Author

We can do the naming discussions afterwards, I would also be okay with groupWith, but I understand the argument that it does not really match the grouped method which returns a map.

@markus1189 markus1189 changed the title WIP: Import splitWith from strawman-contrib Import splitWith from strawman-contrib Jun 18, 2018
@markus1189
Copy link
Author

I finished importing the code and it works.

The only thing remaining is to decide on a good name. Any thoughts? Also see discussion in https://github.com/scala/collection-strawman/issues/241

@julienrf
Copy link
Collaborator

I’m closing this one in favor of #4

@julienrf julienrf closed this Jul 30, 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