-
Notifications
You must be signed in to change notification settings - Fork 204
use go version 1.24.2 #1812
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
use go version 1.24.2 #1812
Conversation
bump |
@xuan-cao-swi is it possible to have this reviewed? |
Hi @biddyfiddy, Sorry for late responses. Actually, there are other places in the collector (https://github.com/open-telemetry/opentelemetry-lambda/tree/main/collector) where we use Go version |
thanks! i updated the other go.mod files. |
idea: it could be great to automate this kind of bump thanks to a GitHub action? |
Thanks @biddyfiddy, even though caret version is used, can you also update the version here: https://github.com/open-telemetry/opentelemetry-lambda/blob/main/.github/workflows/release-layer-collector.yml#L36 |
This might be better as an improvement outside of this PR. My intentions are to quickly bump the version so we can use it in our downstream dependencies. |
done |
Sure! If that sounds good to you @serkan-ozal I can open an issue and draft a PR for this. The less manual work we need to do to stay evergreen the better is it IMO :) |
Hi @maxday, sure, sounds good. let's go ahead! |
Thanks for taking care of every comment @biddyfiddy |
#1802