-
Notifications
You must be signed in to change notification settings - Fork 14
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, Witherable and Align instances #14
Comments
Hi! I'm your new maintainer. :) Is it possible to define the instance outside enummapset? Do we expose enough internals? Are the definitions very long? I think I'd rather have another package that depends on enummapset and adds a lot of instances. Some people would not depend on that new package anyway, but just copy-paste only the instances they need, but that's fine. OTOH, they can't copy-paste enummapset easily, so I'd keep its deps small. What do you think? |
It is possible but I mean that has all the usual downsides of orphan instances. I suppose we could also ask the maintainers of the classes to depend on |
A long shot, but surely try talking with maintainers of these packages and perhaps link back to this issue. Talking and being aware of each other is always a good idea. Regarding orphans, it's unfortunate, but currently that's a common thing with many other packages. |
I will create those issues now. Perhaps the long term solution is to convince people that We really do need a proper long term language-level solution for orphans though. |
Agreed on all accounts. Thanks for that. Perhaps separate |
Slightly related: I noticed http://hackage.haskell.org/package/regex-tdfa defines its own |
I have found the above to be extremely useful for generically working with various types of Maps. Between them you basically get filtering, unioning and intersecting in a convenient and generic way.
I realize this means depending on
witherable
,these
andsemigroupoids
. But alas the current Haskell orphan instances situation means that there isn't really a good way around that.The text was updated successfully, but these errors were encountered: