Skip to content

Upgrade versions#248

Merged
whywaita merged 4 commits intomasterfrom
feat/upgrade-versions-202512
Dec 8, 2025
Merged

Upgrade versions#248
whywaita merged 4 commits intomasterfrom
feat/upgrade-versions-202512

Conversation

@whywaita
Copy link
Copy Markdown
Owner

@whywaita whywaita commented Dec 5, 2025

No description provided.

@whywaita whywaita requested a review from Copilot December 8, 2025 03:45
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR upgrades the go-github library from v47 to v80, updates Go from version 1.23 to 1.25, and pins GitHub Actions to specific commit SHAs. The upgrade includes adapting to API changes in the newer go-github version, such as switching from deprecated functions to new methods and updating type signatures.

Key Changes:

  • Upgraded go-github from v47 to v80 across all Go source files
  • Updated Go version from 1.23 to 1.25 in go.mod and Dockerfile
  • Replaced deprecated github.String() with github.Ptr() and adapted to new API patterns
  • Pinned all GitHub Actions to specific commit SHAs with version comments

Reviewed changes

Copilot reviewed 22 out of 26 changed files in this pull request and generated no comments.

Show a summary per file
File Description
go.mod Updated Go version to 1.25 and upgraded dependencies including go-github to v80
Dockerfile Updated base image to golang:1.25
pkg/gh/github.go Replaced deprecated NewEnterpriseClient() with NewClient().WithEnterpriseURLs()
pkg/gh/jwt.go Adapted token expiration handling to use GetTime() method
pkg/gh/runner.go Updated ListOptions to ListRunnersOptions with nested structure
pkg/starter/starter.go Replaced github.String() with github.Ptr()
internal/util/util.go Switched from golang.org/x/exp/rand to math/rand/v2
.github/workflows/*.yaml Pinned all GitHub Actions to specific commit SHAs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@whywaita whywaita merged commit 287e4c6 into master Dec 8, 2025
5 checks passed
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.

2 participants