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

fix #5 #6

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix #5 #6

wants to merge 2 commits into from

Conversation

xuwei-k
Copy link
Contributor

@xuwei-k xuwei-k commented Apr 17, 2017

No description provided.

@cvogt
Copy link
Owner

cvogt commented Apr 17, 2017

Would it make sense to replace the traversable distinctBy by a version that works for streams? Or maybe somehow anchor it into the collections hierarchy in a way that the strict version is not applicable to Stream? Otherwise a wrong import can turn halting code into non halting code.

@cvogt
Copy link
Owner

cvogt commented Apr 17, 2017

(will have more time to look at this after Scala days and promise to also release a new version soon after)

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.

None yet

2 participants