-
-
Notifications
You must be signed in to change notification settings - Fork 585
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 github.com/google/go-github to v52 #4915
Conversation
Download the artifacts for this pull request:
See Testing a PR |
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.
My intent was that you just upgrade to current go-github. Please don't specify 52 in there. Maybe there's something I don't understand? Just using go-github should get you the right version, and it will be locked until we upgrade it again.
go get -u github.com/google/go-github
- no changes to imports should be required.
That does not work with the new modules feature of go see https://github.com/google/go-github#installation or we have to disable modules which I would not recommend in the first place. |
OK, and I see the v52 is what they suggest. Don't know what they could be saying about modules being enabled; we never used it any time in its history without modules enabled. |
This looks fine to me then and can be merged after |
I manually tested |
As far as I understand modules is a feature since 1.13 and enabled by default. But not all packages support it by default that's why some packages are requested without the versin suffix. |
The Issue
The used package github.com/google/go-github is very outdated and must be updated.
How This PR Solves The Issue
This PR updates the package and makes minor changes in the code because of some breaking changes.
Manual Testing Instructions
Automated Testing Overview
Related Issue Link(s)
Release/Deployment Notes