Skip to content

Conversation

lasarojc
Copy link
Contributor

Partially addresses #174

The SDK uses Go 1.18 from their release that integrates with v0.34, but they should upgrade


PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments

@lasarojc lasarojc requested a review from a team as a code owner February 20, 2023 14:04
@julienrbrt
Copy link
Contributor

julienrbrt commented Feb 20, 2023

Hi, cannot upgrade as it will render our point version consensus breaking due to changes in go 1.19.
Could you stay in go 1.18 on the v0.34.x line?
EDIT: discussing this on slack.

@lasarojc lasarojc mentioned this pull request Feb 21, 2023
Copy link
Contributor

@sergio-mena sergio-mena left a comment

Choose a reason for hiding this comment

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

Based on the latest discussion on Slack. I believe we can merge this now.

Copy link
Contributor

@thanethomson thanethomson left a comment

Choose a reason for hiding this comment

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

Could we please add a changelog entry and a note in our upgrading guidelines for this?

@lasarojc lasarojc merged commit d7600ce into v0.34.x Feb 24, 2023
@lasarojc lasarojc deleted the lasarojc/v0.34.x/174-go-1.19 branch February 24, 2023 11:49
@lasarojc lasarojc self-assigned this Feb 24, 2023
@lasarojc
Copy link
Contributor Author

Could we please add a changelog entry and a note in our upgrading guidelines for this?

Done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants