Upgrade to GHC 9.8.4#4574
Upgrade to GHC 9.8.4#4574rhendric merged 9 commits intopurescript:masterpurescript/purescript:masterfrom
Conversation
|
If this gets merged soon after the previous one so that there aren't intermediate commits, then okay I guess; but either way, please let's not have any more GHC upgrades for a year (or a couple years, given current pace of PS development?) at least. It really throws off the |
Agreed! I would have upgraded to 9.8 immediately, but HLS was broken there for quite some time and it wasn't clear if they were gonna fix it or just move on to 9.10. The good thing about 9.8 is that it's the current Stackage LTS version and the most up-to-date GHC that is not expected to get any further releases (https://endoflife.date/ghc). So this should be stable and serve us well for quite a while 👍 |
|
I think this PR is ready for review @rhendric and @purefunctor! |
|
Thanks for your feedback @rhendric! I applied all your suggestions and it should be good to go now. |
rhendric
left a comment
There was a problem hiding this comment.
Thank you for the new tests! If you would roll back these last two very detail-y things that got altered in the shuffle (just to keep the git blames clean) you've got my approval.
|
Thanks! |
|
What about upgrading to GHC 9.10.2. That looks to be the current most stable release of GHC as of this May 2025. |
|
I'm trying to rebase to this version, and HLS (in VSCode/WSL) fails to run: Executing Has anyone run HSL successfully on this upgrade? |
|
It’s working with HLS for me, but I first had to fully purge all dependency and build files so they are build correctly with the new version! |
What do you mean by purging all dependencies? So far I tried to I wonder why it is saying |
|
After In In So perhaps something should be done about it. |
|
I first mentioned the Cabal breakage in #4568 (review) and if I recall correctly, using 0.8.2.1 causes the built binary to be built with You can update the version bounds for now for local development or maybe use hie.yaml to tell HLS to use |
For clarification, why it it installs 0.8.2.1 if 0.8.2 is required and restricted? And how it is going to be fixed?
Btw I added
Also I have troubles with running HSL on tests/* modules, if src/* and app/* works, tests/* modules give the error on top of the files : UPD: managed to make HLS work in tests/* by adding |
|
I think you might have to run |
|
After setting up ghcup and stack integration HLS started to work fine. |
* Use latest HLint version in CI workflow * Upgrade to latest version of Cheapskate * Allow newer deps for weeder * Use `NonEmpty String` for `directiveStrings`
When I saw that HLS 2.10 was released (which is finally compatible with GHC 9.8 🙌), I couldn't help myself to try to upgrade PureScript to GHC 9.8.4 and luckily it caused less problems than the upgrade to 9.6! 😄
Checklist: