-
Notifications
You must be signed in to change notification settings - Fork 11
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
Derive more #2
base: master
Are you sure you want to change the base?
Derive more #2
Conversation
I have pushed a second commit moving from |
Since we really needed these changes, we have published a fork in Hackage. Sorry for any inconvenience, but otherwise we could not move forward with our project :( |
Thanks for the PR @serras and apologies for the slow reply, I am just catching up on this repo. I don't think I had notifications turned on here. It looks like your PR now includes the fork changes, let me know if you are still interested in merging parts of it. |
I would be happy to prepare a new PR. A question remains, though: this library uses |
It would be great if users could pick only the one they are interested in. I'm not an expert in Haskell tooling though, do you know any good options to support this? |
In theory there is now support for multiple public libraries in a single package, so people could depend on |
Isn't |
Yes, that's true. But often you want to use only one of those, because they come with their own "ecosystem" of surrounding libraries. I think many people would dislike a library if it depended on both |
This PR adds more auto-derivable classes to
TraceT
. These have proved to be useful when usingtracing
as part of the Mu microservice library.