Skip to content

chore: update loader to v0.5.2 - #968

Merged
wdp2020 merged 1 commit into
bytedance:mainfrom
liuq19:fix/update-loader-v0.5.2
Jul 30, 2026
Merged

chore: update loader to v0.5.2#968
wdp2020 merged 1 commit into
bytedance:mainfrom
liuq19:fix/update-loader-v0.5.2

Conversation

@liuq19

@liuq19 liuq19 commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • update the root module dependency from loader v0.5.1 to v0.5.2
  • refresh the loader module checksums
  • consume the independently published Go 1.27-compatible loader from feat(loader): support Go 1.27 #965

This dependency-only change is intentionally separate from the root Go 1.27 build-tag and compatibility changes in #957.

Validation

  • GOWORK=off GOTOOLCHAIN=go1.25.12 go test -race -gcflags="all=-l" ./... -count=1
  • GOWORK=off GOTOOLCHAIN=go1.25.12 SONIC_USE_OPTDEC=1 SONIC_USE_FASTMAP=1 SONIC_ENCODER_USE_VM=1 go test ./... -count=1
  • verified github.com/bytedance/sonic/loader v0.5.2 resolves from the module cache rather than the workspace module

@codecov-commenter

codecov-commenter commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 51.25%. Comparing base (59be92f) to head (98053d8).
⚠️ Report is 84 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #968      +/-   ##
==========================================
- Coverage   51.86%   51.25%   -0.62%     
==========================================
  Files         127      170      +43     
  Lines       10893    14145    +3252     
==========================================
+ Hits         5650     7250    +1600     
- Misses       4920     6504    +1584     
- Partials      323      391      +68     
Flag Coverage Δ
arm 42.88% <ø> (?)
macos-15 44.03% <ø> (?)
ubuntu-24.04-arm 42.88% <ø> (?)
ubuntu-latest 49.72% <ø> (?)
x86 49.72% <ø> (?)

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

☔ View full report in Codecov by Harness.
📢 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.

@liuq19 liuq19 mentioned this pull request Jul 30, 2026
@liuq19
liuq19 marked this pull request as ready for review July 30, 2026 06:03
Copilot AI review requested due to automatic review settings July 30, 2026 06:03
@wdp2020
wdp2020 merged commit a1ed9d8 into bytedance:main Jul 30, 2026
46 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the root Go module’s dependency on github.com/bytedance/sonic/loader to the published v0.5.2 release, aligning the main module with the Go 1.27-compatible loader work referenced in #965 while keeping the change isolated from broader Go 1.27 compatibility work in #957.

Changes:

  • Bump github.com/bytedance/sonic/loader from v0.5.1 to v0.5.2 in go.mod.
  • Refresh go.sum entries to match the updated loader version.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
go.mod Updates the root module requirement for github.com/bytedance/sonic/loader to v0.5.2.
go.sum Updates module checksums to correspond to github.com/bytedance/sonic/loader v0.5.2.

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

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.

4 participants