Skip to content

chore: upgrade to go 1.26, rules_go 0.60#110

Merged
jbedard merged 1 commit intomainfrom
go-125
Mar 13, 2026
Merged

chore: upgrade to go 1.26, rules_go 0.60#110
jbedard merged 1 commit intomainfrom
go-125

Conversation

@jbedard
Copy link
Member

@jbedard jbedard commented Oct 24, 2025

Changes are visible to end-users: no

Test plan

  • Covered by existing test cases

Copilot AI review requested due to automatic review settings October 24, 2025 21:25
Copy link
Contributor

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 project from Go 1.24.5 to Go 1.25.3 and updates rules_go from version 0.56.1 to 0.57.0. The upgrade includes adopting new Go 1.25 standard library features such as strings.CutSuffix, strings.SplitSeq, and slices.Grow to replace older patterns.

Key Changes

  • Updated Go version to 1.25.3 across all go.mod files and go.work
  • Updated rules_go dependency to 0.57.0 in all MODULE.bazel files
  • Refactored code to use Go 1.25 standard library features for improved performance and readability
  • Removed darwin-aarch64 LLVM toolchain configuration

Reviewed Changes

Copilot reviewed 16 out of 17 changed files in this pull request and generated no comments.

Show a summary per file
File Description
runner/pkg/git/gitignore_test.go Replaced HasSuffix/TrimSuffix pattern with strings.CutSuffix and adopted strings.SplitSeq
runner/llvm.MODULE.bazel Removed darwin-aarch64 LLVM toolchain configuration entries
runner/go.mod Updated Go version from 1.24.5 to 1.25.3
runner/MODULE.bazel Updated rules_go version from 0.56.1 to 0.57.0
language/orion/go.mod Updated Go version from 1.24.5 to 1.25.3
language/orion/MODULE.bazel Updated rules_go version from 0.56.1 to 0.57.0
language/kotlin/go.mod Updated Go version from 1.24.5 to 1.25.3
language/kotlin/MODULE.bazel Updated rules_go version from 0.56.1 to 0.57.0
language/js/typescript/tsconfig.go Added slices.Grow for pre-allocation optimization
language/js/node/std_modules.go Replaced strings.Split with strings.SplitSeq
language/js/go.mod Updated Go version from 1.24.5 to 1.25.3
language/js/configure.go Replaced strings.Index pattern with strings.Cut and adopted strings.SplitSeq
language/js/MODULE.bazel Updated rules_go version from 0.56.1 to 0.57.0
go.work Updated Go version from 1.25.0 to 1.25.3
common/go.mod Updated Go version from 1.24.5 to 1.25.3
MODULE.bazel Updated rules_go version from 0.56.1 to 0.57.0

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

@aspect-workflows
Copy link

aspect-workflows bot commented Oct 24, 2025

Test

All tests were cache hits

4 tests (100.0%) were fully cached saving 248ms.


Test

language/js

All tests were cache hits

124 tests (100.0%) were fully cached saving 20s.


Test

language/kotlin

All tests were cache hits

19 tests (100.0%) were fully cached saving 4s.


Test

language/orion

All tests were cache hits

52 tests (100.0%) were fully cached saving 7s.


Test

runner

All tests were cache hits

30 tests (100.0%) were fully cached saving 5s.


Test

runner/e2e/bin

All tests were cache hits

1 test (100.0%) was fully cached saving 93ms.


Buildifier      Gazelle      Gazelle [language/js]      Gazelle [language/kotlin]      Gazelle [language/orion]      Gazelle [runner]      Gazelle [runner/e2e/bin]

Copy link
Contributor

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

Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.


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

@CLAassistant
Copy link

CLAassistant commented Jan 23, 2026

CLA assistant check
All committers have signed the CLA.

Copilot AI review requested due to automatic review settings February 14, 2026 02:49
Copy link
Contributor

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

Copilot reviewed 12 out of 12 changed files in this pull request and generated 8 comments.


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

)

bazel_dep(name = "rules_go", version = "0.59.0")
bazel_dep(name = "rules_go", version = "0.60.0")
Copy link

Copilot AI Feb 14, 2026

Choose a reason for hiding this comment

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

The PR title says this upgrades rules_go to 0.57, but this change sets rules_go to 0.60.0. Please align the PR title/description with the actual version being upgraded to (or adjust the version here if 0.57 was intended).

Copilot uses AI. Check for mistakes.
Copilot AI review requested due to automatic review settings March 13, 2026 03:59
@jbedard
Copy link
Member Author

jbedard commented Mar 13, 2026

@codex

Copy link
Contributor

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

Copilot reviewed 12 out of 17 changed files in this pull request and generated no new comments.


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

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. Nice work!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@jbedard jbedard changed the title chore: upgrade to go 1.25, rules_go 0.57 chore: upgrade to go 1.26, rules_go 0.60 Mar 13, 2026
@jbedard jbedard force-pushed the go-125 branch 2 times, most recently from ab6f026 to 2151fde Compare March 13, 2026 04:08
Copilot AI review requested due to automatic review settings March 13, 2026 04:08
Copy link
Contributor

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

Copilot reviewed 12 out of 17 changed files in this pull request and generated no new comments.


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

@jbedard
Copy link
Member Author

jbedard commented Mar 13, 2026

@codex

@jbedard jbedard marked this pull request as ready for review March 13, 2026 04:19
Copy link
Contributor

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

Copilot reviewed 12 out of 17 changed files in this pull request and generated 1 comment.


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

@jbedard jbedard merged commit d41786e into main Mar 13, 2026
2 of 3 checks passed
@jbedard jbedard deleted the go-125 branch March 13, 2026 04:25
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.

3 participants