Skip to content

chore(deps): bump go-github v87 → v89 and resolve breaking changes#2840

Draft
tommaso-moro wants to merge 1 commit into
mainfrom
tommaso-moro-bump-go-github-v89
Draft

chore(deps): bump go-github v87 → v89 and resolve breaking changes#2840
tommaso-moro wants to merge 1 commit into
mainfrom
tommaso-moro-bump-go-github-v89

Conversation

@tommaso-moro

Copy link
Copy Markdown
Contributor

Summary

Bumps google/go-github from v87 to v89 across the module and resolves the resulting breaking changes. No tool or behavior changes.

Why

go-github v89 adds first-class REST methods for issue dependencies. This PR is the mechanical dependency bump that unblocks that work; the feature itself lands in a stacked follow-up PR. Splitting keeps the ~60-file mechanical import churn separate from the substantive feature diff.

What changed

  • Rewrite all import paths go-github/v87go-github/v89.
  • gists.go: Gists.Create now takes CreateGistRequest by value, and Gists.Edit is renamed to Gists.Update taking UpdateGistRequest.
  • repositories_test.go: adapt to RepositoryRelease fields that became value types in v89 (ID, TagName, Draft).
  • Regenerate third-party license files for the new module path.

MCP impact

  • No tool or API changes

Prompts tested (tool changes only)

N/A — no tool changes.

Security / limits

  • No security or limits impact

Tool renaming

  • I am not renaming tools as part of this PR

Lint & tests

  • Linted locally with ./script/lint
  • Tested locally with ./script/test

Docs

  • Not needed

Bumps google/go-github from v87 to v89 across the module and fixes the
resulting breaking changes. No tool or behavior changes.

- Rewrite all import paths go-github/v87 -> go-github/v89.
- gists.go: Gists.Create now takes CreateGistRequest by value and
  Gists.Edit is renamed to Gists.Update taking UpdateGistRequest.
- repositories_test.go: adapt to RepositoryRelease fields that became
  value types in v89 (ID, TagName, Draft).
- Regenerate third-party license files for the new module path.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant