-
Notifications
You must be signed in to change notification settings - Fork 28
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
GHC 9.2.1 support #116
GHC 9.2.1 support #116
Conversation
Focused on getting it to compile, by commenting out operational parts lavishly. The assumption is GHC 9.2.1 only, a new version, as the concept of separate annotations is now gone. These will be brought back while working through tests.
23 out of 448 tests failed Much easier now that I can use HLS
Down to 17 failing tests
But I think NegApp needs work
Down to 11 failures
Down to 7 test failures
lambda42.hs Structure7.hs
With a modified ghc-exactprint at version 1.5.0 this branch now compiles and passes tests for GHC 9.2.1 |
Should I expect hlint to pass? |
@alanz No need to fix HLint for now. Besides HLint there's some other cleanup we can do but I'm happy to do them after merging this. |
@alanz What do you think? It appears so, but I'm not very up to date wrt GHC 9.2 stuff and I'm not sure whether Alan thinks it's ready to merge. I'll merge it if Alan doesn't respond in the next few days. |
I think it is ready to be merged. And has been for ages. |
Focused on getting it to compile, by commenting out operational parts
lavishly.
The assumption is GHC 9.2.1 only, a new version, as the concept of
separate annotations is now gone.
These will be brought back while working through tests.