Uh oh!
There was an error while loading. Please reload this page.
Bump go-github to v73.0.0 - #597
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR upgrades the go-github module from v72 to v73 across the codebase, updates third-party license documents accordingly, and adds the new dependency to go.mod.
- Bump all
github.com/google/go-github/v72/githubimports to v73 - Update third-party license entries for the new v73 dependency
- Add
github.com/google/go-github/v73 v73.0.0togo.mod
Reviewed Changes
Copilot reviewed 33 out of 34 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| third-party-licenses.windows.md | Replace v72 license entry with v73 |
| third-party-licenses.linux.md | Replace v72 license entry with v73 |
| third-party-licenses.darwin.md | Replace v72 license entry with v73 |
| pkg/raw/raw_test.go | Update import path for go-github to v73 |
| pkg/raw/raw.go | Update import path for go-github to v73 |
| pkg/github/tools.go | Update import path for go-github to v73 |
| pkg/github/server_test.go | Update import path for go-github to v73 |
| pkg/github/server.go | Update import path for go-github to v73 |
| pkg/github/secret_scanning_test.go | Update import path for go-github to v73 |
| pkg/github/secret_scanning.go | Update import path for go-github to v73 |
| pkg/github/search_utils.go | Update import path for go-github to v73 |
| pkg/github/search_test.go | Update import path for go-github to v73 |
| pkg/github/search.go | Update import path for go-github to v73 |
| pkg/github/repository_resource_test.go | Update import path for go-github to v73 |
| pkg/github/repository_resource.go | Update import path for go-github to v73 |
| pkg/github/repositories_test.go | Update import path for go-github to v73 |
| pkg/github/repositories.go | Update import path for go-github to v73 |
| pkg/github/pullrequests_test.go | Update import path for go-github to v73 |
| pkg/github/pullrequests.go | Update import path for go-github to v73 |
| pkg/github/notifications_test.go | Update import path for go-github to v73 |
| pkg/github/notifications.go | Update import path for go-github to v73 |
| pkg/github/issues_test.go | Update import path for go-github to v73 |
| pkg/github/issues.go | Update import path for go-github to v73 |
| pkg/github/context_tools_test.go | Update import path for go-github to v73 |
| pkg/github/code_scanning_test.go | Update import path for go-github to v73 |
| pkg/github/code_scanning.go | Update import path for go-github to v73 |
| pkg/github/actions_test.go | Update import path for go-github to v73 |
| pkg/github/actions.go | Update import path for go-github to v73 |
| pkg/errors/error_test.go | Update import path for go-github to v73 |
| pkg/errors/error.go | Update import path for go-github to v73 |
| internal/ghmcp/server.go | Update import alias for go-github to v73 |
| go.mod | Add github.com/google/go-github/v73 v73.0.0 |
| e2e/e2e_test.go | Update import alias for go-github to v73 |
martinnormark
commented
Jun 27, 2025
Noticing I did the same mistake as in #456 (comment)@williammartin - I tried to run it, and it removed all license files and cleared the |
williammartin
commented
Jun 27, 2025
I'd suggest resetting to your first commit and then running |
martinnormark
commented
Jun 27, 2025
Fixed in 978bfa9, it was a local problem with the |
martinnormark
commented
Jul 3, 2025
@williammartin fixed the license and now targeting the correct branch ( |
SamMorrowDrums
commented
Jul 4, 2025
@martinnormark sorry, we backtracked from the next branch 😓 the sync was going to be too hard! |
martinnormark
commented
Jul 4, 2025
Ok, no worries. Rebased my branch now. You may want to change the CONTRIBUTING.md file if the |
SamMorrowDrums
commented
Jul 4, 2025
I did, thanks for your vigilance. I updated it thanks to you. ✨ |
SamMorrowDrums
commented
Jul 4, 2025
Sorry for the churn, just need to update dependabot pkg too and then we can merge. |
Hi @martinnormark , this PR is almost ready to be merged (thank you!) It looks like the dependabot.go import is the only blocker here so if you'd like feel free to go ahead and get it updated and then we can merge this :) if not, we're happy to take it from here |
martinnormark
commented
Jul 14, 2025
Hi @tommaso-moro - fantastic! I'm currently on vacation with limited access to internet, so if you can move it along that would be great. Thanks! |
Uh oh!
There was an error while loading. Please reload this page.
* Bump go-github to v73.0.0 * Clean up go.mod and update licenses * Updated remaining imports to use github package v73 instead of v72 --------- Co-authored-by: Tommaso Moro <37270480+tommaso-moro@users.noreply.github.com> Co-authored-by: tommaso-moro <tommaso-moro@github.com>
Decided to make this upgrade in separate PR, instead of mixing with #470