Skip to content

[BREAKING CHANGE] feat: 适配 go1.25 以及 kitex0.14.1 - #309

Merged
jiuxia211 merged 7 commits into
mainfrom
upgrade/1.25
Aug 24, 2025
Merged

[BREAKING CHANGE] feat: 适配 go1.25 以及 kitex0.14.1#309
jiuxia211 merged 7 commits into
mainfrom
upgrade/1.25

Conversation

@ozline

@ozline ozline commented Aug 23, 2025

Copy link
Copy Markdown
Member

自查 PR 结构

  • PR 标题符合这个格式: <type>(optional scope): <description>

  • 此 PR 标题的描述以用户为导向,足够清晰,其他人可以理解。

  • 我已经对所有 commit 提供了签名(GPG 密钥签名、SSH 密钥签名)

  • 这个 PR 属于强制变更/破坏性更改

如果是,请在 PR 标题中添加 BREAKING CHANGE 前缀,并在 PR 描述中详细说明。

这个 PR 的类型是什么?

refactor

这个 PR 做了什么 / 我们为什么需要这个 PR?

  • 抬升基础版本到 go1.25,同时升级所有依赖包,重点在 kitex 升级到 0.14.2,以及 launch_screen 模块的新版流处理支持
  • Makefile test 更新,适配 1.25(实际上是一个 bug,等 go 官方修复)
  • 修复部分场景下 cache 导致的测试失败,新增测试环境检测

关于 Makefile test 的改动,可以参考:golang/go#75031

(可选)这个 PR 解决了哪个/些 issue?

对 Reviewer 预留的一些提醒

可以重点关注一下pkg/base/client/rpc.go这个文件中的改动

renbaoshuo
renbaoshuo previously approved these changes Aug 23, 2025
@codecov

codecov Bot commented Aug 23, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 6.29921% with 119 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
kitex_gen/launch_screen/k-launch_screen.go 0.00% 56 Missing ⚠️
pkg/cache/cache.go 0.00% 9 Missing ⚠️
pkg/cache/academic/set_scores.go 0.00% 6 Missing ⚠️
pkg/cache/launch_screen/set_launch_screen.go 0.00% 6 Missing ⚠️
pkg/cache/version/add_visit.go 0.00% 6 Missing ⚠️
pkg/cache/classroom/set_common_emptyroom.go 0.00% 3 Missing ⚠️
pkg/cache/classroom/set_exam_room.go 0.00% 3 Missing ⚠️
pkg/cache/classroom/set_xiamen_emptyroom.go 0.00% 3 Missing ⚠️
pkg/cache/common/set_contributor.go 0.00% 3 Missing ⚠️
pkg/cache/common/set_term_info.go 0.00% 3 Missing ⚠️
... and 8 more
@@          Coverage Diff           @@
##             main    #309   +/-   ##
======================================
  Coverage        ?   5.66%           
======================================
  Files           ?     247           
  Lines           ?   28816           
  Branches        ?       0           
======================================
  Hits            ?    1632           
  Misses          ?   27084           
  Partials        ?     100           
Flag Coverage Δ
unittest 5.66% <6.29%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
kitex_gen/academic/academic.go 0.00% <ø> (ø)
...ex_gen/academic/academicservice/academicservice.go 19.86% <100.00%> (ø)
kitex_gen/academic/academicservice/client.go 0.00% <ø> (ø)
kitex_gen/academic/academicservice/server.go 0.00% <ø> (ø)
kitex_gen/academic/k-academic.go 0.00% <ø> (ø)
kitex_gen/classroom/classroom.go 1.28% <ø> (ø)
...gen/classroom/classroomservice/classroomservice.go 34.93% <100.00%> (ø)
kitex_gen/classroom/classroomservice/client.go 0.00% <ø> (ø)
kitex_gen/classroom/classroomservice/server.go 0.00% <ø> (ø)
kitex_gen/classroom/k-classroom.go 0.00% <ø> (ø)
... and 47 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jiuxia211
jiuxia211 merged commit 79ea1a3 into main Aug 24, 2025
10 checks passed
@jiuxia211
jiuxia211 deleted the upgrade/1.25 branch August 24, 2025 13:27
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