-
Notifications
You must be signed in to change notification settings - Fork 44
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 IdentityT #120
Comments
I'd prefer to wait until we have a concrete use case for it, personally. |
I am trying to port hedgehog to PureScript and they use it. That's why I
opened this issue.
However, I can keep that code in the port until we have more use cases than
this.
|
Ah okay, I reckon that probably counts. Can you describe how it's useful for your hedgehog port? |
It is used to create 3 different type class instances and I'm guessing that |
Instance definitions don't really justify the existence of IdentityT though, I think the part that's more relevant here is places where those instances would be used. I think perhaps similarly to how |
Yeah, that would make sense. Similar to how After looking around a bit more, seems like |
Yep, that works for me. |
I can submit a PR for this. However, what should the file's module name be? |
Yep, that conforms to the convention we're using in this library. |
For context, this is included in the Haskell 'transformers' library
The text was updated successfully, but these errors were encountered: