Skip to content

Conversation

@flc1125
Copy link
Contributor

@flc1125 flc1125 commented Mar 12, 2024

I found an official example, I see the official example for the number of CPU cores for comparison, so I made a reference to the official adjustment.

References:

@codecov
Copy link

codecov bot commented Mar 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.18%. Comparing base (3dc1cd6) to head (a136e90).
Report is 24 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3883      +/-   ##
==========================================
- Coverage   99.21%   99.18%   -0.03%     
==========================================
  Files          42       43       +1     
  Lines        3182     2701     -481     
==========================================
- Hits         3157     2679     -478     
+ Misses         17       12       -5     
- Partials        8       10       +2     
Flag Coverage Δ
?
-race ∅ <ø> (?)
-tags "sonic avx" 99.17% <ø> (?)
-tags go_json 99.17% <ø> (?)
-tags nomsgpack 99.17% <ø> (?)
go-1.18 99.11% <ø> (-0.01%) ⬇️
go-1.19 99.18% <ø> (-0.03%) ⬇️
go-1.20 99.18% <ø> (-0.03%) ⬇️
go-1.21 99.18% <ø> (-0.03%) ⬇️
go-1.22 99.18% <ø> (?)
macos-latest 99.18% <ø> (-0.03%) ⬇️
ubuntu-latest 99.18% <ø> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@flc1125
Copy link
Contributor Author

flc1125 commented Mar 12, 2024

@appleboy As a complement to this: #3878 (comment)

@appleboy appleboy added the test label Mar 14, 2024
@appleboy appleboy added this to the v1.10 milestone Mar 14, 2024
@appleboy appleboy merged commit fd60a24 into gin-gonic:master Mar 14, 2024
@appleboy
Copy link
Member

@flc1125 Thanks.

@flc1125 flc1125 deleted the test-gc branch March 14, 2024 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants