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
Allow compilation with ghc 8.10.7 #51
Conversation
.github/workflows/haskell-ci.yml
Outdated
- name: Install dependencies | ||
run: | | ||
stack build --test --bench --no-run-tests --no-run-benchmarks --only-dependencies |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you want these stack commands to use the stack yaml files, you'll have to specify it on cli,
something like stack build .... --stack-yaml stack-${{ matrix.ghc }}.yaml
Otherwise they'll just use the stack.yaml
.
Also if you want to use the ghc installed by haskell/actions/setup@v1
you should add --system-ghc
to all the stack commands (otherwise stack will install its own GHC and ignore the ghc installation provided by that setup action.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can see it's downloading ghc-8.8.3 in 8.10.7 job
which is because the stack.yaml specifies lts-16.2 which uses ghc 8.8.3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for pointing this out. I did not realise this was happening.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like to use lazy approach to this. Rather than being forced to check everything all the time use the workflow where it's hard to miss the problem. If you want to make something build with new version of compiler, start by replicating compilation error in ci. Only then attempt to fix it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add cli arguments to use --system-ghc
and --stack-yaml stack-${{ matrix.ghc }}.yaml
to all stack invocations. You'll also probably need to rename stack.yaml to stack-8.8.4.yaml and use some resolver that depends on 8.8.4 (e.g. latest 17: https://www.stackage.org/lts-17.15)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, forgot to click "request changes"
It's possible you'll no longer be able to build with GHC 8.6.5, due to constraints introduced in 75599c2 In that case I'd recommend simply dropping ghc 8.6.5. |
Or alternatively you can just add appropriate extra-deps to stack-8.6.5.yaml. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thank you!
I love the consistent formatting 👍
Need to allow GHC 8.10.7.
Current build on
master
: