-
Notifications
You must be signed in to change notification settings - Fork 65
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
Re-remove re-exports, minimal bounds, CI #108
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
herculean 💪
Re:
I've asked about whether we dealt with other re-exports or not: I assume you only meant those three in your item, but I had to check. This is a necessary preliminary to other work.
Deferring to @chessai, but I believe those three were the offending re-exports.
-- This is a SafeHaskell safeguard (pun intended) to explicitly declare the API contract of `mtl` | ||
-- GHC versions before 7.4 were hopelessly broken or incapable of SafeHaskell | ||
if impl(ghc >= 7.4) | ||
default-extensions: Safe |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removal of default-extensions: Safe
may allow some modules to became unsafe unnoticed. Please add it back.
This completes multiple required milestones in #86:
Control.Monad
,Control.Monad.Fix
andData.Monoid
are gone.cabal-install
3.0.transformers-0.5.6
.@emilypi I've asked about whether we dealt with other re-exports or not: I assume you only meant those three in your item, but I had to check. This is a necessary preliminary to other work.