Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support GHC 9.10 #149

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Support GHC 9.10 #149

wants to merge 5 commits into from

Conversation

jhrcek
Copy link

@jhrcek jhrcek commented Jun 28, 2024

Hello @zliu41

I see you created ghc-9.10 branch but the last commit on it is from a month ago.
I'd like to help moving this forward, because I'd like to get hlint plugin enabled in haskell language server for ghc 9.10. But that is blocked by hlint upgrade to ghc 9.10, which in turn is blocked by upgrading apply-refact to ghc 9.10 😄

Working off your branch, I managed to reduce the amount of test failures from 249 to 79 doing two small changes.
Do you intend to get back to this?
If not, I can try my best over the weekend to move this forward..

@zliu41
Copy link
Collaborator

zliu41 commented Jun 29, 2024

@jhrcek Thanks for the PR! I'm on PTO at the moment, will try to get back to this as soon as I can. In the meantime, if you are motivated to fix more tests, feel free to do so!

@zliu41
Copy link
Collaborator

zliu41 commented Jul 1, 2024

2b11376 fixed a few test failures.

This also fixes a few, such as Bracket2 and Default75:

- let new' = transferEntryDP old new
+ let new' = setEntryDP (transferEntryDP old new) (GHC.SameLine 1)

However it causes many new test failures. I'm going to look into this tomorrow.

@ndmitchell
Copy link
Contributor

The fact that this doesn't work with GHC 9.10 means HLint no longer gets replacements with GHC 9.10, which is a little bit of a shame. But I can document it if that is the expected outcome.

@zliu41
Copy link
Collaborator

zliu41 commented Feb 3, 2025

@ndmitchell Yes unfortunately ghc-exactprint doesn't quite work with GHC 9.10.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants