-
-
Notifications
You must be signed in to change notification settings - Fork 398
Remove hie-compat
#4613
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
base: master
Are you sure you want to change the base?
Remove hie-compat
#4613
Conversation
d330291
to
3c9d053
Compare
Yeah, I think this is a good idea. In principle, I think we'd like to have some kind of compatibility layer library, but in practice I think it's just spread out everywhere. |
There is |
b6b5ecb
to
ec910c4
Compare
Any other opinions? (I don't feel like dragging this along for too long, but maybe there is a way forward?) |
I think @wz1000 is most likely to have a real opinion? |
9d3f4d8
to
11ab93f
Compare
While removing references to GHC 9.4, I realized that some parts of HLS are referring to even older versions of GHC. For example, `hie-compat` is a compatibility library backporting support of Haskell IDE Engine (HIE) features to older versions of GHC. Since GHC 9.2, `hie-compat` only re-exported definitions already present in the `ghc` library, and so, is essentially obsolete. FYI: We still have `hie-compat` in the dependency graph, because some libraries (e.g., `hiedb`) are using it.
11ab93f
to
be8be55
Compare
It seems there are no more opinions. I sgguest reviewing/merging or closing this PR. |
I think we just need an opinion from @wz1000 |
After removing support for GHC 9.4, I was in the mood of cleaning up a bit more. While removing references to GHC 9.4, I realized that some parts of HLS are referring to even older versions of GHC.
Now, some people may hate me for that, but: I completely removed the now obsolete
hie-compat
library.hie-compat
is a compatibility library backporting support of Haskell IDE Engine (HIE) features to older versions of GHC. Since GHC 9.2,hie-compat
only re-exported definitions already present in theghc
library, and so, is essentially obsolete.FYI: We still have
hie-compat
in the dependency graph, because some libraries (e.g.,hiedb
) are using it.This may be a bit harsh, but let me know what you think!