Gabriella439 / Haskell-MMorph-Library

Monad morphisms
BSD 3-Clause "New" or "Revised" License
47 stars 26 forks source link

Module Control.Monad.Trans.List is deprecated #49

Closed istathar closed 3 years ago

istathar commented 4 years ago

Another deprecation for you to consider.

/tmp/stack6/mmorph-1.1.3/src/Control/Monad/Morph.hs:83:1: warning: [-Wdeprecations]
Module `Control.Monad.Trans.List' is deprecated:
   This transformer is invalid on most monads
    |
 83 | import qualified Control.Monad.Trans.List          as L
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

This was from GHC 8.6.5 compiling against Stackage snapshot lts-14.6.

AfC

turion commented 3 years ago

Replace it with https://hackage.haskell.org/package/list-t? Or https://hackage.haskell.org/package/pipes?

turion commented 3 years ago

Given that list-t and pipes have a dependency on mmorph already, probably the instance should be deleted.

Gabriella439 commented 3 years ago

Fix is up here: https://github.com/Gabriel439/Haskell-MMorph-Library/pull/63