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

Remove GHC <= 9.0 from the support window #1778

Merged
merged 3 commits into from
Sep 12, 2024
Merged

Conversation

tchoutri
Copy link
Contributor

@tchoutri tchoutri commented Aug 29, 2024

GHC <= 9.0 is removed from the support window. The lowest version supported in CI becomes GHC 9.2.8, released on the 26/05/2023

This grants us the ability to use -XGHC2021 as default language, as well as -XOverloadedRecordDot.

@tchoutri tchoutri force-pushed the remove-support-for-8.10.7 branch 5 times, most recently from 4a12fe4 to a9ab6df Compare August 30, 2024 09:39
@tchoutri
Copy link
Contributor Author

@ysangkok I'd love a review :)

changelog.d/1778 Outdated
prs: #1778

description {
GHC <= 8.10.7 is removed from the support window. The lowest version supported in CI becomes GHC 9.2.1, released on the 29/10/2021.
Copy link
Contributor

Choose a reason for hiding this comment

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

If the lower base bound is 4.16.4.0, then that corresponds to 9.2.5, not 9.2.1. So I think this should say 9.2.5

Copy link
Contributor Author

Choose a reason for hiding this comment

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

you're right, actually latest patch version of 9.2 (9.2.8) should be the one we should aim

@tchoutri tchoutri force-pushed the remove-support-for-8.10.7 branch 2 times, most recently from 601a00c to 287d50b Compare September 1, 2024 22:33
Copy link
Contributor

@ysangkok ysangkok left a comment

Choose a reason for hiding this comment

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

PR description should be updated to reflect that 9.0.x series is also excluded in this PR

servant-client-core/servant-client-core.cabal Outdated Show resolved Hide resolved
servant-client-core/servant-client-core.cabal Outdated Show resolved Hide resolved
servant-swagger/servant-swagger.cabal Outdated Show resolved Hide resolved
GHC <= 8.10.7 is removed from the support window. The lowest version
supported in CI becomes GHC 9.2.1, released on the 29/10/2021.

This grants us the ability to use `-XGHC2021` as default language, as
well as `-XOverloadedRecordDot`.
@tchoutri tchoutri changed the title Remove GHC <= 8.10.7 from the support window Remove GHC <= 9.0 from the support window Sep 5, 2024
@ysangkok
Copy link
Contributor

ysangkok commented Sep 5, 2024

Missing minimum bytestring bumps for:

  • servant-auth/servant-auth-server/servant-auth-server.cabal
  • servant-auth/servant-auth-client/servant-auth-client.cabal
  • servant-quickcheck/servant-quickcheck.cabal

Missing minimum container bumps for:

  • servant-auth/servant-auth-client/servant-auth-client.cabal
  • servant-auth/servant-auth/servant-auth.cabal
  • servant/servant.cabal

@tchoutri
Copy link
Contributor Author

@ysangkok alright, I believe I have answered your reviews.

@tchoutri tchoutri merged commit 87d449c into master Sep 12, 2024
6 checks passed
@tchoutri tchoutri deleted the remove-support-for-8.10.7 branch September 12, 2024 12:47
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