-
Notifications
You must be signed in to change notification settings - Fork 32
List _all_ operations available via implicit enrichment in README #59
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
Conversation
I have signed the Scala CLA. |
@erikvanoosten are you sure listing all methods on each collection is a good idea? I mean, most of the new methods are inherited from IMHO this would make it harder to identify the methods added to specific subhierarchies of collections. |
No, I am not sure of that :) What initiated this pull request is that I was unaware of Iterator.splitBy. I assumed Iterator was not enriched by this library and wrote it myself. Then when I wanted to contribute it, and looked in the source it was already there! What might be better is to just have the list of enriched traits and leave out the operations, OR |
Another option is to list the operations, and for each (group of) operation mention for which traits it is implemented. |
I think keeping this list up to date will be complicated. What about just linking to each decorator documentation? BTW, I’m surprised to see |
Well, |
I removed the implied operations and appended |
Thanks Erik! |
In README.md this removes the incomplete list with operations available via implicit enrichment and replaces it with a link per decorator and a plain text list of all available operators per decorator.