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

basic GitHub Actions; fix tests; GHC 9 still failing #28

Merged
merged 4 commits into from
Feb 2, 2022

Conversation

peterbecich
Copy link

@peterbecich peterbecich commented Jan 22, 2022

@peterbecich
Copy link
Author

peterbecich commented Jan 24, 2022

The tests are failing to build, but #21 might help

@arianvp
Copy link
Member

arianvp commented Jan 24, 2022

One of the issues is simply a missing instance for Fragment a :> api. Which should be the same as the QueryParam a instance. Fixes #29

@peterbecich
Copy link
Author

peterbecich commented Jan 30, 2022

To fix building with GHC 9, there are some dependencies which may need to be updated.

For GHC 8, everything seems to be good now: https://github.com/peterbecich/servant-ekg/runs/4994048104?check_suite_focus=true

@peterbecich peterbecich changed the title basic GitHub Actions basic GitHub Actions; fix GHC 8 tests Jan 30, 2022
@peterbecich peterbecich changed the title basic GitHub Actions; fix GHC 8 tests basic GitHub Actions; fix tests; GHC 9 still failing Jan 30, 2022
@arianvp
Copy link
Member

arianvp commented Jan 31, 2022

Alright; shall we turn GHC 9 support into a new PR and merge tthis as is?

.github/workflows/ci.yml Outdated Show resolved Hide resolved
@peterbecich peterbecich requested a review from arianvp February 1, 2022 23:00
@arianvp arianvp merged commit bda075f into haskell-servant:master Feb 2, 2022
@peterbecich peterbecich deleted the basic-github-actions branch February 17, 2022 08:12
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.

2 participants