-
Notifications
You must be signed in to change notification settings - Fork 27
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
feat(go): support go 1.23 and delve 1.23.1 #140
feat(go): support go 1.23 and delve 1.23.1 #140
Conversation
henrymcconville
commented
Nov 25, 2024
•
edited
Loading
edited
- Upgrade Go to 1.23
- Upgrade Delve to 1.23.1
- Update Delve patch based on Delve 1.23 codebase
- Add tests for go 1.21-1.23
- Remove unused tests for Go 1.13-1.17
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
dbb3da9
to
a006f77
Compare
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.
Thanks for updating this! Can you rebase so that this PR is included and the integration tests will pass? Thanks!
a006f77
to
47b98a6
Compare
@plumpy done! |
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.
Thanks!!
Thanks a lot! Could you possibly make a release? When will Skaffold (https://github.com/GoogleContainerTools/skaffold) pick up the changes and make a release? |
Yep, I created a release (accidentally jumped straight from 1.4 to 1.6... oops) and Skaffold should start using it automatically. |