Skip to content

ci(GHC): Drop GHC version 8 and add 9.6, 9.8, 9.10 as supported #89

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

Merged
merged 16 commits into from
May 9, 2025

Conversation

davidnbr
Copy link
Contributor

No description provided.

@davidnbr davidnbr force-pushed the db/bump_ghc_version_support branch from 96f6062 to 76959ae Compare April 17, 2025 20:36
@davidnbr davidnbr changed the title ci(GHC): Drop GHC version 8 and add 9.6, 9.8, 9.10, 9.12 as supported ci(GHC): Drop GHC version 8 and add 9.6, 9.8, 9.10 as supported Apr 24, 2025
@davidnbr davidnbr requested a review from sestrella April 24, 2025 20:31
@davidnbr davidnbr self-assigned this Apr 24, 2025
@davidnbr davidnbr requested a review from DavidMazarro May 1, 2025 19:51
@davidnbr davidnbr requested a review from sestrella May 8, 2025 23:20
DavidMazarro
DavidMazarro previously approved these changes May 9, 2025
Copy link
Member

@DavidMazarro DavidMazarro left a comment

Choose a reason for hiding this comment

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

LGTM!

@DavidMazarro DavidMazarro self-requested a review May 9, 2025 09:58
@DavidMazarro DavidMazarro dismissed their stale review May 9, 2025 09:59

I didn't realize the version bumped was a major one

Copy link
Member

@DavidMazarro DavidMazarro left a comment

Choose a reason for hiding this comment

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

I left a comment about the version bump. Also, maybe you want to add the latest GHC version (9.12.1) as part of the tests, as suggested by @jpvillaisaza and @sestrella in Slack?

@@ -375,7 +377,7 @@ instance ToJSON Notifier where
defaultNotifier :: Notifier
defaultNotifier = Notifier
{ notifierName = "rollbar-client"
, notifierVersion = "1.1.0"
, notifierVersion = T.pack (showVersion Paths.version)
Copy link
Member

Choose a reason for hiding this comment

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

@davidnbr great improvement 💯

@davidnbr davidnbr merged commit 591c1d5 into main May 9, 2025
9 checks passed
@davidnbr davidnbr deleted the db/bump_ghc_version_support branch May 9, 2025 14:18
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.

3 participants