apache / dubbo-go

Go Implementation For Apache Dubbo .
https://dubbo.apache.org/
Apache License 2.0
4.69k stars 917 forks source link

[3.1] cherry-pick ioutil replace to release-3.1 #2597

Closed dongjiang1989 closed 7 months ago

dongjiang1989 commented 7 months ago

ref: https://github.com/apache/dubbo-go/pull/2593#issuecomment-1943077815

@justxuewei PTAL

sonarcloud[bot] commented 7 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

codecov-commenter commented 7 months ago

Codecov Report

Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (6dba60a) 44.97% compared to head (25c592c) 44.96%.

:exclamation: Current head 25c592c differs from pull request most recent head 8e6504c. Consider uploading reports for the commit 8e6504c to get more accurate results

Files Patch % Lines
protocol/jsonrpc/server.go 0.00% 4 Missing :warning:
config/config_loader_options.go 0.00% 1 Missing and 1 partial :warning:
config/tls_config.go 0.00% 2 Missing :warning:
protocol/jsonrpc/http.go 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## release-3.1 #2597 +/- ## =============================================== - Coverage 44.97% 44.96% -0.02% =============================================== Files 273 273 Lines 18469 18469 =============================================== - Hits 8306 8304 -2 Misses 9272 9272 - Partials 891 893 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

dongjiang1989 commented 7 months ago

Hey @justxuewei , can you merge it?