Skip to content

ci: reduce Go matrix to representative versions#1601

Merged
xushiwei merged 1 commit intomainfrom
xgopilot/issue-1600-1770091111
Feb 4, 2026
Merged

ci: reduce Go matrix to representative versions#1601
xushiwei merged 1 commit intomainfrom
xgopilot/issue-1600-1770091111

Conversation

@xgopilot
Copy link
Contributor

@xgopilot xgopilot bot commented Feb 3, 2026

Requested by @luoliwoshang

Summary

  • Reduced Go version matrix from ["1.21.13", "1.22.12", "1.23.6", "1.24.2"] to ["1.21.13", "1.24.2"]
  • Updated both llgo and hello jobs in .github/workflows/llgo.yml

Rationale

Most Go-related issues fall into two categories separated by Go 1.22 (alias support boundary):

  • Pre–Go 1.22 (1.21.13) — no alias support
  • Go 1.22+ (1.24.2) — alias supported, latest stable

This change:

  • Reduces CI jobs from 8 to 4 for both llgo and hello jobs (50% reduction)
  • Preserves coverage of the key semantic boundary at Go 1.22
  • Enables better scalability as simulator tests are added

Fixes #1600

Keep only Go 1.21.13 (pre-Go 1.22) and Go 1.24.2 (latest stable)
to cover the key semantic boundary at Go 1.22 alias support.

This reduces CI jobs while preserving meaningful coverage.

Fixes #1600

Generated with [codeagent](https://github.com/qbox/codeagent)
Co-authored-by: luoliwoshang <51194195+luoliwoshang@users.noreply.github.com>
os: [ubuntu-latest, macos-latest]
llvm: [19]
go: ["1.21.13", "1.22.12", "1.23.6", "1.24.2"]
go: ["1.21.13", "1.24.2"]
Copy link
Member

Choose a reason for hiding this comment

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

Hello job Current maybe not loss lots time,maybe keep the matrix here

@codecov
Copy link

codecov bot commented Feb 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.06%. Comparing base (38d9fa2) to head (20183b6).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1601   +/-   ##
=======================================
  Coverage   91.06%   91.06%           
=======================================
  Files          45       45           
  Lines       11981    11981           
=======================================
  Hits        10911    10911           
  Misses        894      894           
  Partials      176      176           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@xushiwei xushiwei merged commit ff8c462 into main Feb 4, 2026
37 checks passed
@xgopilot xgopilot bot deleted the xgopilot/issue-1600-1770091111 branch February 4, 2026 23:30
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.

ci: Keep only representative Go versions in CI matrix

3 participants