Skip to content
This repository has been archived by the owner on Oct 7, 2020. It is now read-only.

Hlint 2.1.22 #1270

Merged
merged 15 commits into from
May 26, 2019
Merged

Hlint 2.1.22 #1270

merged 15 commits into from
May 26, 2019

Conversation

alanz
Copy link
Collaborator

@alanz alanz commented May 25, 2019

Bring in hlint-2.1.22, which uses ghc-lib-parser when it can, instead of haskell-src-exts.

It does not support GHC < 8.4, so those still use the older hlint.

alanz added 12 commits May 14, 2019 23:15
Expecting GHC 8.2.2 version to fail.
There is a problem in the unit tests, getting feedback for a haskell
file with a parse error in it.

See ndmitchell/hlint#637
Still get segfault
It still fails, now hanging on the call to applyRefactorings.
This uses ghc-lib-parser-0.20190523, which avoids a segfault present
in prior versions when used with haskell-ide-engine.
ghc-lib-parser takes a long time to build the first time.
@alanz alanz requested review from lukel97 and wz1000 May 25, 2019 16:43
test.sh Outdated

# stack --stack-yaml=stack-8.6.4.yaml test haskell-ide-engine:unit-test --test-arguments "--match \"ApplyRefactPlugin\""

# stack --stack-yaml=stack.yaml test haskell-ide-engine:func-test --test-arguments "--match \"/FunctionalCodeActions/code actions/hlint suggestions/provides 3.8 code actions/\""
Copy link
Collaborator

Choose a reason for hiding this comment

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

Did you mean to add this file?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I added it to help the ghc-lib-parser guys track down the problem. I guess it can be removed again.

@alanz alanz merged commit fa58b5c into haskell:master May 26, 2019
@alanz alanz added this to the 2019-05 milestone Jun 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants