-
Notifications
You must be signed in to change notification settings - Fork 7
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
Bump Dependency Bounds & Fix Test Suite #14
Bump Dependency Bounds & Fix Test Suite #14
Conversation
Also fixes #12 |
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.
Looks great. Thank you.
Thanks & welcome back! Can we get this merged in? I got hackage maintainer permission while you were away so I could keep the dependencies up to date. If you're willing to make me a collaborator on this repo, I can make sure the version bounds in here are kept up to date as well. I haven't poked deep enough to make any actual useful code changes though 😅 |
Updated for newly released deepseq 1.6 |
Hi @prikhi, happy to make you a maintainer on Hackage. What's your username on Hackage? |
Thanks! Had the hackage trustees add me as a hackage maintainer already when dependencies broke. |
Thank you for doing this. |
No problem, happy to keep this working, few of my projects use it via transitive dependencies. Thank you for building it! |
Extension of #13
Bump deepseq to support 1.5.0.0, fix compilation error in test suite.
Tested builds with
stack build --resolver nightly-2024-02-04
&cabal build -w ghc-9.8 --constraint='deepseq==1.5.0.0
Test-suite passed with
stack run --resolver ngihtly-2024-02-04
&cabal run -w ghc-9.8 --constraint='deepseq==1.5.0.0'
intest/
directory.