Skip to content

Remove noncanonical definitions. - #15

Merged
Yuras merged 2 commits into
Yuras:masterfrom
monad-of-no-return:no-return-refactor
Jul 31, 2026
Merged

Remove noncanonical definitions.#15
Yuras merged 2 commits into
Yuras:masterfrom
monad-of-no-return:no-return-refactor

Conversation

@L0neGamer

Copy link
Copy Markdown
Contributor

This appeases the -Wnoncanonical-monad-instances warning. A future GHC release will treat noncanonical definitions as errors. See haskell/core-libraries-committee#328.

@Yuras

Yuras commented Jul 30, 2026

Copy link
Copy Markdown
Owner
  • I restored the CI, could you please rebase to make sure package compiles with older versions of ghc.
  • I guess this makes the package not working with older versions of base, right? Could you please add a lower version bound if applicable.

This appeases the `-Wnoncanonical-monad-instances` warning. A future GHC
release will treat noncanonical definitions as errors. See
haskell/core-libraries-committee#328.
@L0neGamer
L0neGamer force-pushed the no-return-refactor branch from bbc4803 to a72cc15 Compare July 31, 2026 19:15
@L0neGamer

Copy link
Copy Markdown
Contributor Author

What is your support window? This should support all the way back to GHC 7.10. I've amended the base bounds but I cannot test it locally, 7.10 doesn't seem to work for me.

@Yuras

Yuras commented Jul 31, 2026

Copy link
Copy Markdown
Owner

What is your support window?

It's "whatever CI is configured to build with". Previously it included ghc-7.6: https://github.com/Yuras/scanner/blob/master/.travis.yml, but looks like the latest cabal doesn't work with ghc-7 anymore: https://github.com/Yuras/scanner/actions/runs/30544000177/job/90875413894. So now it's ghc-8.0+.

Anyway, the CI is green, I'll merge the PR shortly. Thank you!

@Yuras
Yuras merged commit 144381e into Yuras:master Jul 31, 2026
13 checks passed
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.

3 participants