Skip to content

feat(triple) Remove old triple implementation - #3451

Open
Alanxtl wants to merge 5 commits into
apache:developfrom
Alanxtl:remove_old_triple
Open

feat(triple) Remove old triple implementation#3451
Alanxtl wants to merge 5 commits into
apache:developfrom
Alanxtl:remove_old_triple

Conversation

@Alanxtl

@Alanxtl Alanxtl commented Jun 25, 2026

Copy link
Copy Markdown
Member

Description

Fixes #3093 #2894

!! BREAKING CHANGE: no more support old-triple and old-triple fallback
!! BREAKING CHANGE: no more support the stub code generated by old version of protoc-gen-go-triple, need to regenerat stub code by latest protoc-gen-go-triple

  1. complete remove old triple
  2. update protoc-gen-go-triple to remove generated old triple
  3. update dubbogo-cli to remove old triple demonstration
  4. no more depend on https://github.com/dubbogo/triple
  5. no more depend on https://github.com/dubbogo/grpc-go

Checklist

  • I confirm the target branch is develop
  • Code has passed local testing
  • I have added tests that prove my fix is effective or that my feature works

@Alanxtl Alanxtl linked an issue Jun 25, 2026 that may be closed by this pull request
2 tasks
@Alanxtl

Alanxtl commented Jun 25, 2026

Copy link
Copy Markdown
Member Author

ci 失败是因为 sample 还依赖 old triple
已提 pr 修复 apache/dubbo-go-samples#1116

@codecov-commenter

codecov-commenter commented Jun 25, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 55.44%. Comparing base (60d1c2a) to head (1173234).
⚠️ Report is 890 commits behind head on develop.

Files with missing lines Patch % Lines
protocol/triple/triple.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3451      +/-   ##
===========================================
+ Coverage    46.76%   55.44%   +8.67%     
===========================================
  Files          295      453     +158     
  Lines        17172    34746   +17574     
===========================================
+ Hits          8031    19264   +11233     
- Misses        8287    13961    +5674     
- Partials       854     1521     +667     

☔ 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.

@Alanxtl

Alanxtl commented Jun 25, 2026

Copy link
Copy Markdown
Member Author

@CAICAIIs @NeverENG help review this

@Alanxtl
Alanxtl force-pushed the remove_old_triple branch from 0ae472f to 1173234 Compare July 13, 2026 07:11
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deprecate github.com/dubbogo/grpc-go usage [FEATURE] remove old triple protocol Delete protocol/dubbo3 and protocol/grpc grpc-go版本升级

2 participants