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

Add GHC 9.8.1 to the cabal CI #775

Merged
merged 13 commits into from
Nov 12, 2023
Merged

Add GHC 9.8.1 to the cabal CI #775

merged 13 commits into from
Nov 12, 2023

Conversation

mihaimaruseac
Copy link
Owner

@mihaimaruseac mihaimaruseac commented Oct 14, 2023

Description of the PR

Checklist

Part of #760

@toku-sa-n
Copy link
Collaborator

@mihaimaruseac Can I override this PR to support GHC 9.8, or should I create another PR?

@mihaimaruseac
Copy link
Owner Author

Up to you. I didn't get more time to look into it for the past few days :(

@toku-sa-n toku-sa-n marked this pull request as draft October 22, 2023 02:24
mihaimaruseac and others added 5 commits November 11, 2023 10:27
Signed-off-by: Mihai Maruseac <mihai.maruseac@gmail.com>
Signed-off-by: Mihai Maruseac <mihai.maruseac@gmail.com>
@mihaimaruseac
Copy link
Owner Author

I rebased to remove the merges from master back into the branch as those made understanding the changes harder

@mihaimaruseac mihaimaruseac marked this pull request as ready for review November 12, 2023 06:13
@mihaimaruseac
Copy link
Owner Author

Thank you! This seems to be going through now

@mihaimaruseac mihaimaruseac enabled auto-merge (squash) November 12, 2023 06:21
@mihaimaruseac
Copy link
Owner Author

I cannot approve this since I started the PR, but this LGTMs. I enabled auto-merge, if you can approve it will merge automatically once tests finish. Otherwise, I'll force merge

@toku-sa-n toku-sa-n self-requested a review November 12, 2023 06:37
@mihaimaruseac mihaimaruseac merged commit 39091d2 into master Nov 12, 2023
23 checks passed
@mihaimaruseac mihaimaruseac deleted the add-new-ghc branch November 12, 2023 06:41
@mihaimaruseac
Copy link
Owner Author

Thank you! I'll cut a new release tomorrow

@mihaimaruseac mihaimaruseac mentioned this pull request Nov 12, 2023
@toku-sa-n
Copy link
Collaborator

Thank you and sorry for the long delay. I'll fix the remaining warnings when I have time.

@mihaimaruseac
Copy link
Owner Author

No worries about the delay, I delayed this for too long myself.

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