-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add Apply and Traverse for SortedMap #1966
Comments
Wouldn't we get conflicting instances because SortedMap is a subtype of Map? |
So I tried this and it seemed to work just fine. Not sure why I thought it wouldn't work. So I think we could get all the instances for SortedMap, that already exist for Map, but require an instance of Order[A] right? Could potentially be relevant for #1831 |
Maybe we need to make it higher priority? Or maybe not. Subtyping and implicit resolution confuses me sometimes with what is and is not ambiguous. |
We should probably also add instances for SortedSet while we're at it. |
+1 |
Actually there can't be any instances for SortedSet, as that'd require an implicit Order [A] instance, when there is no actual A. |
When there are multiple instances for the target type |
see #1964 for more discussion.
The text was updated successfully, but these errors were encountered: