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

Make sure to give the correct DynFlags to the recompilation checker #1459

Merged
merged 6 commits into from
Mar 2, 2021

Conversation

pepeiborra
Copy link
Collaborator

We were passing the wrong DynFlags here, causing most modules with a LANGUAGE pragma to fail the recompilation check and have their interface files regenerated.

Copy link
Member

@jneira jneira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks for the fix

@pepeiborra pepeiborra added the merge me Label to trigger pull request merge label Feb 28, 2021
We were passing the wrong DynFlags here, causing most modules with a LANGUAGE pragma
to fail the recompilation check and have their interface files regenerated
@wz1000
Copy link
Collaborator

wz1000 commented Mar 1, 2021

I thought I already fixed this :(

Well, I did in haskell/ghcide#891 but the rest of the changes turned out to be incorrect.

@mergify mergify bot merged commit a70711a into master Mar 2, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
merge me Label to trigger pull request merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants