Skip to content

Comments

chore: bump minimum Go version to 1.23.0#3113

Merged
dnwe merged 4 commits intomainfrom
dnwe/bump-minimum-go
Mar 3, 2025
Merged

chore: bump minimum Go version to 1.23.0#3113
dnwe merged 4 commits intomainfrom
dnwe/bump-minimum-go

Conversation

@dnwe
Copy link
Collaborator

@dnwe dnwe commented Mar 2, 2025

Dependencies are all starting to move to go 1.23.0 going forwards, so we'll sadly have to start bumping Sarama's go directive more aggressively as well in order to keep up. The Go maintainers have unfortunately decided that:

A module’s go line must declare a version greater than or equal to the
go version declared by each of the modules listed in require
statements.

and

Before Go 1.21, Go toolchains treated the go line as an advisory
requirement: if builds succeeded the toolchain assumed everything
worked, and if not it printed a note about the potential version
mismatch. Go 1.21 changed the go line to be a mandatory requirement
instead.

So we really have no option here, especially with the golang.org/x modules bumping to 1.23.0 as soon as 1.24.0 was released, we can no longer continue to support older Go toolchains.

Also bump CI testing to use the "oldstable" and "stable" aliases for N-1 and N going forwards as we're unlikely to want to test with anything else.

dnwe and others added 4 commits March 2, 2025 17:16
Dependencies are all starting to move to go 1.23.0 going forwards, so
we'll sadly have to start bumping Sarama's go directive more
aggressively as well in order to keep up. The Go maintainers have
unfortunately decided that:

> A module’s go line must declare a version greater than or equal to the
> go version declared by each of the modules listed in require
> statements.

and

> Before Go 1.21, Go toolchains treated the go line as an advisory
> requirement: if builds succeeded the toolchain assumed everything
> worked, and if not it printed a note about the potential version
> mismatch. Go 1.21 changed the go line to be a mandatory requirement
> instead.

So we really have no option here, especially with the golang.org/x
modules bumping to 1.23.0 as soon as 1.24.0 was released, we can no
longer continue to support older Go toolchains.

Also bump CI testing to use the "oldstable" and "stable" aliases for N-1
and N going forwards as we're unlikely to want to test with anything
else.

Signed-off-by: Dominic Evans <dominic.evans@uk.ibm.com>
Signed-off-by: Dominic Evans <dominic.evans@uk.ibm.com>
Signed-off-by: Dominic Evans <dominic.evans@uk.ibm.com>
Signed-off-by: Dominic Evans <8060970+dnwe@users.noreply.github.com>
@dnwe dnwe force-pushed the dnwe/bump-minimum-go branch from ab675c1 to 4343972 Compare March 3, 2025 15:07
Copy link
Collaborator

@hindessm hindessm left a comment

Choose a reason for hiding this comment

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

LGTM

@dnwe dnwe merged commit dd9aba8 into main Mar 3, 2025
16 checks passed
@dnwe dnwe deleted the dnwe/bump-minimum-go branch March 3, 2025 15:40
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