haskell / mtl

The Monad Transformer Library
http://www.haskell.org/haskellwiki/Monad_Transformers
Other
366 stars 65 forks source link

Switch back to haskell-ci #97

Closed phadej closed 2 years ago

phadej commented 2 years ago

Add GHC-7 jobs, add base lower bound

chessai commented 2 years ago

What is the justification for haskell-ci? The ci.yml becomes much more verbose.

Additionally, do you think we should move back to supporting 7.x?

phadej commented 2 years ago

Additionally, do you think we should move back to supporting 7.x?

There is no reason to not to. They just work. EDIT: dropping support will make transformers-compat (which sometimes depend on mtl) story complicated (transformers-compat depend only on old mtl). I don't want to know how. Also drop of GHC-7 wasn't mentioned in the changelog nor reflected in the bounds.

The ci.yml becomes much more verbose.

You need to care only about the first lines:

# This GitHub workflow config has been generated by a script via
#
#   haskell-ci 'github' '--installed=+all -transformers' '--haddock-jobs=>=8.0' 'mtl.cabal'
#
# To regenerate the script (for example after adjusting tested-with) run
#
#   haskell-ci regenerate
#
# For more information, see https://github.com/haskell-CI/haskell-ci

It also does more checks, which is good.

chessai commented 2 years ago

Thanks!