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

CI: Add support for GHC 8.10, stack and live-server testing #156

Merged
merged 7 commits into from
Jun 24, 2020

Conversation

donatello
Copy link
Member

No description provided.

.github/workflows/haskell.yml Outdated Show resolved Hide resolved
@donatello donatello force-pushed the ci branch 6 times, most recently from 695e339 to d720f9b Compare June 18, 2020 18:50
@donatello donatello force-pushed the ci branch 4 times, most recently from c7bdfe3 to f346c34 Compare June 19, 2020 21:45
Comment on lines 26 to 31
- ghc: 8.6
os: windows-latest
experimental: false
- ghc: 8.10
os: windows-latest
experimental: false
Copy link
Member Author

Choose a reason for hiding this comment

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

These are the same by default right? Don;t think we need these.

Comment on lines 25 to 35
- ghc: 8.6
os: windows-latest
experimental: false
- ghc: 8.10
os: windows-latest
experimental: false

# Appears to be buggy to build in windows with ghc 8.4 and 8.8
- ghc: 8.4
os: windows-latest
experimental: true
Copy link
Member Author

Choose a reason for hiding this comment

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

In stack we are only building with 8.8, so we dont need these as well.

@krisis krisis force-pushed the ci branch 3 times, most recently from 78a500c to 5b9ebeb Compare June 23, 2020 23:35
@donatello donatello requested a review from harshavardhana June 24, 2020 00:10
@donatello donatello dismissed harshavardhana’s stale review June 24, 2020 00:10

Minio is running with 4 disks now.

Copy link

@kannappanr kannappanr left a comment

Choose a reason for hiding this comment

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

LGTM

@donatello
Copy link
Member Author

Windows builds that fail due to an unrelated error have been marked experimental, so CI can enforce that other build configurations must succeed. We will be merging this now, and making a release subsequently.

@donatello donatello merged commit a3538aa into minio:master Jun 24, 2020
@donatello donatello deleted the ci branch June 24, 2020 17:35
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.

4 participants