-
Notifications
You must be signed in to change notification settings - Fork 86
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
Support GHC 9.4 #298
Support GHC 9.4 #298
Conversation
#else | ||
w# | ||
#endif | ||
)))) |
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.
Can this be a CPP macro instead? Using #if’s everywhere feels messy to me.
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.
This is the limit of my CPP knowledge - I'd love it if there was an easier way!
We still need: * cborg * serialise which support 9.4.2. Both are addressed in well-typed/cborg#298
We still need: * cborg * serialise which support 9.4.2. Both are addressed in well-typed/cborg#298
We still need: * cborg * serialise which support 9.4.2. Both are addressed in well-typed/cborg#298
@FinleyMcIlwaine @bgamari any chance to merge and release this please? |
1940993
to
c966382
Compare
* Remove 8.0.2 and 8.2.2 from tested with * Replace 8.10.1 with 8.10.7 in tested with * Add allow-newer constraint on text for store-core to make benchmarks compile
c966382
to
8da1a09
Compare
Thanks @FinleyMcIlwaine, could you please possibly make a Hackage release any time soon? |
@Bodigrim Done, finally 🙂 Apologies for the delay! |
Huge thanks! |
Fixes #291
This is a very stupid PR, where I use
CPP
and then do_f
in the 9.4 case to see what to put in. But the tests all pass, so 🎉 🤷🏻