Skip to content

use go 1.20 -cover support#10361

Merged
ndeloof merged 1 commit intodocker:v2from
ndeloof:go120_cover
Mar 10, 2023
Merged

use go 1.20 -cover support#10361
ndeloof merged 1 commit intodocker:v2from
ndeloof:go120_cover

Conversation

@ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Mar 10, 2023

What I did
Use go1.20 -cover to generate coverage data while running e2e tests
based on https://go.dev/blog/integration-test-coverage

(not mandatory) A picture of a cute animal, if possible in relation to what you did
image

@ndeloof ndeloof force-pushed the go120_cover branch 2 times, most recently from 95050d7 to d4af59c Compare March 10, 2023 10:09
@codecov
Copy link

codecov bot commented Mar 10, 2023

Codecov Report

Patch coverage has no change and project coverage change: -20.77 ⚠️

Comparison is base (6a0398d) 74.18% compared to head (87a17dc) 53.41%.

Additional details and impacted files
@@             Coverage Diff             @@
##               v2   #10361       +/-   ##
===========================================
- Coverage   74.18%   53.41%   -20.77%     
===========================================
  Files           2      104      +102     
  Lines         275     8911     +8636     
===========================================
+ Hits          204     4760     +4556     
- Misses         60     3634     +3574     
- Partials       11      517      +506     

see 106 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@ndeloof ndeloof force-pushed the go120_cover branch 2 times, most recently from cc9fbdd to f8110cd Compare March 10, 2023 10:49
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
@ndeloof ndeloof merged commit 85ddfde into docker:v2 Mar 10, 2023
@ndeloof ndeloof deleted the go120_cover branch March 10, 2023 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants