Skip to content

Conversation

alice-yyds
Copy link
Contributor

No description provided.

Marina-Sakai and others added 23 commits February 27, 2025 10:28
Co-authored-by: alice <90381261+alice-yyds@users.noreply.github.com>
Co-authored-by: YangruiEmma <yangrui.emma@bytedance.com>
#### What type of PR is this?
<!--
Add one of the following kinds:

build: Changes that affect the build system or external dependencies
(example scopes: gulp, broccoli, npm)
ci: Changes to our CI configuration files and scripts (example scopes:
Travis, Circle, BrowserStack, SauceLabs)
docs: Documentation only changes
feat: A new feature
optimize: A new optimization
fix: A bug fix
perf: A code change that improves performance
refactor: A code change that neither fixes a bug nor adds a feature
style: Changes that do not affect the meaning of the code (white space,
formatting, missing semi-colons, etc)
test: Adding missing tests or correcting existing tests
chore: Changes to the build process or auxiliary tools and libraries
such as documentation generation
-->

#### Check the PR title.
<!--
The description of the title will be attached in Release Notes, 
so please describe it from user-oriented, what this PR does / why we
need it.
Please check your PR title with the below requirements:
-->
- [ ] This PR title match the format: \<type\>(optional scope):
\<description\>
- [ ] The description of this PR title is user-oriented and clear enough
for others to understand.
- [ ] Attach the PR updating the user documentation if the current PR
requires user awareness at the usage level. [User docs
repo](https://github.com/cloudwego/cloudwego.github.io)


#### (Optional) Translate the PR title into Chinese.


#### (Optional) More detailed description for this PR(en: English/zh:
Chinese).
<!--
Provide more detailed info for review(e.g., it's recommended to provide
perf data if this is a perf type PR).
-->
en:
zh(optional): 添加 Go 1.24 单元测试

#### (Optional) Which issue(s) this PR fixes:
<!--
Automatically closes linked issue when PR is merged.
Eg: `Fixes #<issue number>`, or `Fixes (paste link of issue)`.
-->

#### (Optional) The PR that updates user documentation:
<!--
If the current PR requires user awareness at the usage level, please
submit a PR to update user docs. [User docs
repo](https://github.com/cloudwego/cloudwego.github.io)
-->
* added Listener() method for network transport
* check Listener() in IsRunning()
Change-Id: Ia4b838c591072648db3c62d2919972838d0fed99
Change-Id: Ia4b838c591072648db3c62d2919972838d0fed99

#### What type of PR is this?
<!--
Add one of the following kinds:

build: Changes that affect the build system or external dependencies
(example scopes: gulp, broccoli, npm)
ci: Changes to our CI configuration files and scripts (example scopes:
Travis, Circle, BrowserStack, SauceLabs)
docs: Documentation only changes
feat: A new feature
optimize: A new optimization
fix: A bug fix
perf: A code change that improves performance
refactor: A code change that neither fixes a bug nor adds a feature
style: Changes that do not affect the meaning of the code (white space,
formatting, missing semi-colons, etc)
test: Adding missing tests or correcting existing tests
chore: Changes to the build process or auxiliary tools and libraries
such as documentation generation
-->

#### Check the PR title.
<!--
The description of the title will be attached in Release Notes, 
so please describe it from user-oriented, what this PR does / why we
need it.
Please check your PR title with the below requirements:
-->
- [ ] This PR title match the format: \<type\>(optional scope):
\<description\>
- [ ] The description of this PR title is user-oriented and clear enough
for others to understand.
- [ ] Attach the PR updating the user documentation if the current PR
requires user awareness at the usage level. [User docs
repo](https://github.com/cloudwego/cloudwego.github.io)


#### (Optional) Translate the PR title into Chinese.


#### (Optional) More detailed description for this PR(en: English/zh:
Chinese).
<!--
Provide more detailed info for review(e.g., it's recommended to provide
perf data if this is a perf type PR).
-->
en:
zh(optional):

#### (Optional) Which issue(s) this PR fixes:
<!--
Automatically closes linked issue when PR is merged.
Eg: `Fixes #<issue number>`, or `Fixes (paste link of issue)`.
-->

#### (Optional) The PR that updates user documentation:
<!--
If the current PR requires user awareness at the usage level, please
submit a PR to update user docs. [User docs
repo](https://github.com/cloudwego/cloudwego.github.io)
-->
@alice-yyds alice-yyds requested review from a team as code owners April 9, 2025 03:15
Copy link

codecov bot commented Apr 9, 2025

Codecov Report

Attention: Patch coverage is 95.98214% with 9 lines in your changes missing coverage. Please review.

Project coverage is 72.80%. Comparing base (7c3893c) to head (1b33abb).
Report is 26 commits behind head on main.

Files with missing lines Patch % Lines
pkg/protocol/header.go 88.88% 5 Missing ⚠️
internal/testutils/testutils.go 93.47% 3 Missing ⚠️
pkg/app/client/client.go 90.90% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1325      +/-   ##
==========================================
- Coverage   73.15%   72.80%   -0.36%     
==========================================
  Files         138      139       +1     
  Lines       13533    16244    +2711     
==========================================
+ Hits         9900    11826    +1926     
- Misses       3036     3823     +787     
+ Partials      597      595       -2     

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

HeyJavaBean
HeyJavaBean previously approved these changes Apr 9, 2025
ppzqh
ppzqh previously approved these changes Apr 9, 2025
@HeyJavaBean HeyJavaBean dismissed stale reviews from ppzqh and themself April 9, 2025 03:20

The merge-base changed after approval.

@HeyJavaBean HeyJavaBean merged commit d02d673 into main Apr 9, 2025
27 of 28 checks passed
@HeyJavaBean HeyJavaBean deleted the release-v0.9.7 branch April 9, 2025 03:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

10 participants