Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

## [Codecov](https://app.codecov.io/gh/alibaba/nacos/pull/11856?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=alibaba) Report #12590

Closed
A6976 opened this issue Sep 3, 2024 · 3 comments
Labels
status/invalid This doesn't seem right

Comments

@A6976
Copy link

A6976 commented Sep 3, 2024

          ## [Codecov](https://app.codecov.io/gh/alibaba/nacos/pull/11856?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=alibaba) Report

Attention: Patch coverage is 15.03532% with 842 lines in your changes are missing coverage. Please review.

Project coverage is 66.61%. Comparing base (64b9e8d) to head (7f84376).
Report is 13 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##             develop   #11856      +/-   ##
=============================================
- Coverage      67.81%   66.61%   -1.20%     
- Complexity      8893     8945      +52     
=============================================
  Files           1237     1254      +17     
  Lines          40443    41399     +956     
  Branches        4286     4422     +136     
=============================================
+ Hits           27425    27580     +155     
- Misses         11046    11805     +759     
- Partials        1972     2014      +42     
Files Coverage Δ
...acos/client/config/impl/ConfigTransportClient.java 86.11% <ø> (ø)
...n/java/com/alibaba/nacos/api/common/Constants.java 53.84% <0.00%> (-4.49%) ⬇️
...emote/response/ConfigBatchFuzzyListenResponse.java 0.00% <0.00%> (ø)
...mote/response/FuzzyListenNotifyChangeResponse.java 0.00% <0.00%> (ø)
...remote/response/FuzzyListenNotifyDiffResponse.java 0.00% <0.00%> (ø)
...m/alibaba/nacos/config/server/model/CacheItem.java 76.38% <0.00%> (-3.33%) ⬇️
...java/com/alibaba/nacos/client/utils/ParamUtil.java 86.17% <62.50%> (-5.88%) ⬇️
...onfig/server/configuration/ConfigCommonConfig.java 72.72% <40.00%> (-27.28%) ⬇️
...mote/request/AbstractFuzzyListenNotifyRequest.java 0.00% <0.00%> (ø)
...acos/config/server/service/ConfigCacheService.java 75.34% <0.00%> (-1.09%) ⬇️
... and 17 more

... and 32 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 64b9e8d...7f84376. Read the comment docs.

Originally posted by @codecov-commenter in #11856 (comment)

@KomachiSion
Copy link
Collaborator

No useful information in issue.

@KomachiSion KomachiSion closed this as not planned Won't fix, can't repro, duplicate, stale Sep 4, 2024
@KomachiSion KomachiSion added the status/invalid This doesn't seem right label Sep 4, 2024
@A6976
Copy link
Author

A6976 commented Sep 21, 2024

Details

@A6976
Copy link
Author

A6976 commented Sep 21, 2024

  • [ ]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

2 participants