Skip to content
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

Load plugins in the frontend and honour pragmas and cabal #20

Merged
merged 2 commits into from
Jun 16, 2019

Conversation

fendor
Copy link

@fendor fendor commented May 24, 2019

This is a follow up to #19 which reverts changes of #19.
@mpickering fixed the problem of #19, that plugin pragmas in source files are not being honoured.
Now ghc-mod correctly detects the plugin param in *.cabal and source pragma.

This reverts the previous changes since they are not necessary anymore.
Problem: In the logs of ghc-mod, you still see the type errors, but they are not reported to hie. May lead to confusion for people that have the logs constantly open.

@Avi-D-coder
Copy link

@alanz @fendor is something blocking this?

@fendor
Copy link
Author

fendor commented Jun 2, 2019

Just time, will be done soon.

@Avi-D-coder
Copy link

@fendor So If I build hie with this pr Polysemy plugin works?

@fendor
Copy link
Author

fendor commented Jun 2, 2019

Yes, the example project worked fine.

@infinisil
Copy link

Would love for this to be in the next HIE release

@alanz
Copy link
Owner

alanz commented Jun 14, 2019

It will be, my task for tomorrow

@alanz alanz merged commit d050fac into alanz:thinned-api Jun 16, 2019
alanz added a commit to alanz/haskell-ide-engine that referenced this pull request Jun 16, 2019
@isovector
Copy link

isovector commented Aug 31, 2019

Is it feasible to upstream this into hie-bios? haskell/hie-bios#16

EDIT: sorry, thought this was haskell-ide-engine, not ghc-mod

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

Successfully merging this pull request may close these issues.

5 participants