Skip to content

fix(leak): init() check for goroutine profile might fail under pressu… - #73

Merged
fredbi merged 1 commit into
go-openapi:masterfrom
fredbi:fix/leak-init-check
Mar 7, 2026
Merged

fix(leak): init() check for goroutine profile might fail under pressu…#73
fredbi merged 1 commit into
go-openapi:masterfrom
fredbi:fix/leak-init-check

Conversation

@fredbi

@fredbi fredbi commented Mar 7, 2026

Copy link
Copy Markdown
Member

…re: now wait a bit

Change type

Please select: 🆕 New feature or enhancement|🔧 Bug fix'|📃 Documentation update

Short description

Fixes

Full description

Checklist

  • I have signed all my commits with my name and email (see DCO. This does not require a PGP-signed commit
  • I have rebased and squashed my work, so only one commit remains
  • I have added tests to cover my changes.
  • I have properly enriched go doc comments in code.
  • I have properly documented any breaking change.
…re: now wait a bit

Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
@codecov

codecov Bot commented Mar 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 18.18182% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.24%. Comparing base (87b4b45) to head (1c7de48).
⚠️ Report is 2 commits behind head on master.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
internal/leak/leak.go 18.18% 8 Missing and 1 partial ⚠️

❌ Your patch status has failed because the patch coverage (18.18%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
- Coverage   91.29%   91.24%   -0.05%     
==========================================
  Files          91       91              
  Lines       11925    11929       +4     
==========================================
- Hits        10887    10885       -2     
- Misses        824      830       +6     
  Partials      214      214              

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

@fredbi
fredbi merged commit da68477 into go-openapi:master Mar 7, 2026
20 of 21 checks passed
@fredbi
fredbi deleted the fix/leak-init-check branch March 7, 2026 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant