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 Control.Monad.Trans for mtl-2.3 compat #1471

Merged
merged 1 commit into from
Mar 2, 2023

Conversation

TeofilC
Copy link
Contributor

@TeofilC TeofilC commented Feb 1, 2023

I needed to make this change to have this module compile with GHC-9.6.1-alpha1.
I see that there is already an mtl-2.3 compat branch merged, so I'm not sure why this wasn't picked up there.
Perhaps this was being re-exported by something else?

@parsonsmatt
Copy link
Collaborator

Please add a changelog entry and version bump and I'll be happy to get this merged + released. Thank you!

mtl-2.3.1 no longer re-exports this from another module
@TeofilC
Copy link
Contributor Author

TeofilC commented Feb 1, 2023

Sure I've force pushed adding a changelog entry and a patch version bump.

@parsonsmatt
Copy link
Collaborator

Great thanks!

Super minor request: in the future, for larger PRs, please don't force-push - it makes "view changes since last commit" unusable, which makes reviewing PRs harder. I do a squash+merge to keep the commit history clean.

@TeofilC
Copy link
Contributor Author

TeofilC commented Feb 1, 2023

Sure no worries! I'll keep that in mind

Copy link
Collaborator

@parsonsmatt parsonsmatt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

@parsonsmatt parsonsmatt merged commit 2e08f3b into yesodweb:master Mar 2, 2023
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.

None yet

2 participants