-
Notifications
You must be signed in to change notification settings - Fork 7
Add testing support on ghc >= 9.4 #21
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
Conversation
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.
LGTM
cabal: ["3.6"] | ||
cache-version: ["2022-10-25"] | ||
|
||
ghc-version: ["8.6", "8.8", "8.10", "9.4", "9.6", "9.8", "9.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.
I'd like to add support for 9.10
as well, but I think it would be better to handle it in a separate PR.
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.
I have tested locally and 9.10.1 works on my computer. Haven't tried running the tests in CI or locally though.
@jimbob88 Thank you for your contribution! 👍 |
This pull request adds testing on 9.6, 9.8, 9.12 and drops support for testing on 9.2 and 9.0. I also removed 9.0 and 9.2 from the supported versions in the cabal package.
However, when being run in the tests, it also adds the current commit hash as follows (see here):
Due to this change, I updated the tests to use a regex and extract each function that appears in the call stack. It then checks that every function that should appear in the stack trace, appears in that order.
This pull request fixes the tests on 9.4 (the code worked but the tests couldn't run due to a version constraint).