-
Notifications
You must be signed in to change notification settings - Fork 63
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 go sdk dependencies. #168
Conversation
This reverts commit b54571a.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but imo we should use the newest versions.
@@ -24,7 +24,7 @@ jobs: | |||
strategy: | |||
fail-fast: true | |||
matrix: | |||
go: [1.16.15, 1.17.13, 1.18.9, 1.19.4] | |||
go: [1.18.9, 1.19.4] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't we consider the newest stable versions ?
go: [1.18.9, 1.19.4] | |
go: [1.21.13, 1.22.12, 1.23.6, 1.24.0] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the future yes, just this PR is more about clearing Dependabot alerts. But we should definitely prepare tests for new versions as well (in another PR).
@pubnub-release-bot release |
🚀 Release successfully completed 🚀 |
refactor: update go sdk dependencies
update go sdk dependencies to fix GH Dependabot vulnerabilities (end of support for Go 1.16 and 1.17)