Skip to content

MonR: removing mappend method definitions#78

Merged
Lysxia merged 2 commits into
Lysxia:masterfrom
L0neGamer:monr
Jun 24, 2026
Merged

MonR: removing mappend method definitions#78
Lysxia merged 2 commits into
Lysxia:masterfrom
L0neGamer:monr

Conversation

@L0neGamer

Copy link
Copy Markdown
Contributor

As part of my work on haskell/core-libraries-committee#418, I remove the mappend definitions in this package.

@Lysxia

Lysxia commented Jun 22, 2026

Copy link
Copy Markdown
Owner

To deal with the failure with GHC 8.2 I can remove support for it or find a way to turn this -Wmissing-methods off in CI. The failure with GHC 9.2 is unrelated and caused by linear-base (I have opened an issue).

@L0neGamer

Copy link
Copy Markdown
Contributor Author

the 8.2 error can be solved with some CPP if you are willing to put up with it; I did not use some here because I didn't see any here already.

@Lysxia

Lysxia commented Jun 24, 2026

Copy link
Copy Markdown
Owner

At this point I prefer to keep things simple, and I think dropping support for old GHCs is less trouble than maintaining CPP.

@Lysxia Lysxia merged commit 5119e8e into Lysxia:master Jun 24, 2026
11 checks passed
@Lysxia

Lysxia commented Jun 24, 2026

Copy link
Copy Markdown
Owner

Thanks!

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