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

Revert "Use go 1.21 (#344)" #357

Merged
merged 3 commits into from
Oct 19, 2023
Merged

Revert "Use go 1.21 (#344)" #357

merged 3 commits into from
Oct 19, 2023

Conversation

sfc-gh-elinardi
Copy link
Collaborator

@sfc-gh-elinardi sfc-gh-elinardi commented Oct 19, 2023

This reverts commit 04285e2.
Downgrading to keep the minimum go version for importing sansshell at 1.19.

@sfc-gh-elinardi sfc-gh-elinardi marked this pull request as ready for review October 19, 2023 17:42
Copy link
Collaborator

@sfc-gh-srhodes sfc-gh-srhodes left a comment

Choose a reason for hiding this comment

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

Could you add a bit more details to the PR description on why we're doing this?

@sfc-gh-elinardi
Copy link
Collaborator Author

Could you add a bit more details to the PR description on why we're doing this?

Added!

@sfc-gh-elinardi sfc-gh-elinardi merged commit d7d6050 into main Oct 19, 2023
4 checks passed
@sfc-gh-elinardi sfc-gh-elinardi deleted the elinardi-downgrade-go branch October 19, 2023 18:16
sfc-gh-srhodes added a commit that referenced this pull request Apr 3, 2024
This should unblock some currently pending updates. Each major Go release is supported until there are two newer major releases and the newest major version is 1.22, so 1.19 has been unsupported for a while.

We'd previously tried this six months ago and reverted in #357. The cases we had where we wanted an older version should no longer apply.

Prometheus libraries (#415) are the current dependency where newer versions need newer go compilers.
sfc-gh-elinardi pushed a commit that referenced this pull request Apr 3, 2024
* Bump go.mod to 1.21

This should unblock some currently pending updates. Each major Go release is supported until there are two newer major releases and the newest major version is 1.22, so 1.19 has been unsupported for a while.

We'd previously tried this six months ago and reverted in #357. The cases we had where we wanted an older version should no longer apply.

Prometheus libraries (#415) are the current dependency where newer versions need newer go compilers.

* Address go lint errors

* Add in process test changes from #344
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