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

explicitly import forM_ in test; fix tests on GHC 9.6 #82

Closed
wants to merge 1 commit into from

Conversation

raehik
Copy link

@raehik raehik commented Apr 17, 2023

Tests are broken on GHC 9.6 due to an unscoped forM_ usage. It used to be exported by Data.Foldable, and does appear to be present from there on GHC 9.2.7. I can't see where Data.Foldable is imported -- I assume it's re-exported by one of the imports, but I can't tell which.

@raehik
Copy link
Author

raehik commented Apr 17, 2023

My GHC 9.2.7 emits an unused import message for import Control.Monad ( forM_ ). Not sure what to do about that (can we ignore a single instance?)

@raehik
Copy link
Author

raehik commented Apr 17, 2023

Looks like forM_ is imported a few times via the various mtl Control.Monad.{Except,Identity} modules. mtl-2.3 removed those re-exports (changelog).

MichaelXavier added a commit that referenced this pull request Apr 17, 2023
This resolves #82. The code in that MR worked but it produced warnings
in the current default lts for the project. I used the qualified import
trick to resolve the warning. Then the partial matches started failing
in newer GHC versions so I resolved that too. This now builds without
warnings on the project LTS, the newest LTS and nightly.
@MichaelXavier
Copy link
Contributor

Thanks for this MR. I ended up solving it slightly differently to resolve warnings in different GHC versions. It has been released as retry-0.9.3.1

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

Successfully merging this pull request may close these issues.

2 participants