Skip to content
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

No MaybeT #6

Closed
adek05 opened this issue Apr 26, 2013 · 7 comments
Closed

No MaybeT #6

adek05 opened this issue Apr 26, 2013 · 7 comments
Assignees

Comments

@adek05
Copy link

adek05 commented Apr 26, 2013

Why there is no MaybeT transformer in library? There is a implemenation of it here:
http://www.haskell.org/haskellwiki/New_monads/MaybeT
Why not to put it inside?

@ekmett
Copy link
Member

ekmett commented Apr 27, 2013

It is already in transformers and mtl depends on transformers.

There is no class required for MaybeT, however. It is described entirely by MonadPlus

Just import Control.Monad.Trans.Maybe.

@ekmett ekmett closed this as completed Apr 27, 2013
@ghost ghost assigned ekmett Apr 27, 2013
@ekmett
Copy link
Member

ekmett commented Apr 28, 2013

A case might be made that perhaps we should offer a Control.Monad.Maybe to re-export it here though.

@adek05
Copy link
Author

adek05 commented Apr 28, 2013

Actually the case is that it will not offer these lovely instances which avoid have usage of lifting common in transformers library.

@ekmett
Copy link
Member

ekmett commented Apr 28, 2013

If we're missing instances for lifting over MaybeT then I should definitely fix that. It just doesn't have/need a class of its own.

@ekmett
Copy link
Member

ekmett commented Apr 28, 2013

They are all here. e.g. there is already an

instance MonadState s m => MonadState s (MaybeT m)

etc.

@sdroege
Copy link

sdroege commented Dec 15, 2014

They are in the MaybeT package, but the version in transformers does not have them. Compare:
https://hackage.haskell.org/package/MaybeT-0.1.2/docs/Control-Monad-Maybe.html
https://hackage.haskell.org/package/transformers-0.4.2.0/docs/Control-Monad-Trans-Maybe.html

For example the above mentioned
MonadState s m => MonadState s (MaybeT m)
is not there in the transformers version.

@sdroege
Copy link

sdroege commented Dec 15, 2014

Ah nevermind. The instances are on MonadState (etc) in mtl.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants