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

Update to go 1.21 #195

Merged
merged 3 commits into from
Mar 22, 2024
Merged

Update to go 1.21 #195

merged 3 commits into from
Mar 22, 2024

Conversation

monotypical
Copy link
Contributor

@monotypical monotypical commented Mar 11, 2024

What

Use go v1.21

Why

A recent go buildpack update is dropping support for go 1.20, so updating to 1.21 will let us update this buildpack

How to review

Ensure that the aiven broker deploys okay to a dev environment and passes tests

Who can review

Current PaaS team members


🚨⚠️ Please do not merge this pull request via the GitHub UI ⚠️🚨

@dark5un dark5un force-pushed the 187200424-update-to-go-1-21 branch 7 times, most recently from 777b0ce to 6a7537d Compare March 13, 2024 11:35
Signed-off-by: Alice Lee <alice.lee@digital.cabinet-office.gov.uk>
@dark5un dark5un force-pushed the 187200424-update-to-go-1-21 branch from 6a7537d to c06a6b9 Compare March 13, 2024 11:42
@monotypical monotypical marked this pull request as ready for review March 13, 2024 12:18
Copy link
Contributor

@jackjoy-gds jackjoy-gds left a comment

Choose a reason for hiding this comment

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

ok

@dark5un dark5un merged commit 32d267e into main Mar 22, 2024
3 checks passed
@dark5un dark5un deleted the 187200424-update-to-go-1-21 branch March 22, 2024 16:19
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