Closed crazyStar00 closed 2 years ago
@crazyStar00 Aha, you have sharp eyes.
Merging #1208 (2079aa5) into master (e05233e) will increase coverage by
0.02%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #1208 +/- ##
============================================
+ Coverage 71.58% 71.60% +0.02%
- Complexity 830 832 +2
============================================
Files 408 408
Lines 17224 17224
Branches 2684 2684
============================================
+ Hits 12329 12333 +4
+ Misses 3530 3524 -6
- Partials 1365 1367 +2
Impacted Files | Coverage Δ | |
---|---|---|
.../java/com/alipay/sofa/rpc/config/ServerConfig.java | 92.99% <ø> (ø) |
|
...ofa/rpc/message/bolt/BoltFutureInvokeCallback.java | 71.69% <0.00%> (-7.55%) |
:arrow_down: |
.../com/alipay/sofa/rpc/context/RpcInvokeContext.java | 81.48% <0.00%> (-0.93%) |
:arrow_down: |
...lipay/sofa/rpc/message/AbstractResponseFuture.java | 56.14% <0.00%> (-0.88%) |
:arrow_down: |
...om/alipay/sofa/rpc/server/triple/TripleServer.java | 75.00% <0.00%> (+1.11%) |
:arrow_up: |
.../alipay/sofa/rpc/metrics/lookout/RpcLookoutId.java | 88.88% <0.00%> (+1.58%) |
:arrow_up: |
...java/com/alipay/sofa/rpc/module/LookoutModule.java | 85.71% <0.00%> (+33.33%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update e05233e...2079aa5. Read the comment docs.
错别字
Motivation:
Explain the context, and why you're making that change. To make others understand what is the problem you're trying to solve.
Modification:
Describe the idea and modifications you've done.
Result:
Fixes #.
If there is no issue then describe the changes introduced by this PR.