Skip to content

fix(resilience4j): remove intermediate error wrapping clashing with resilience4j filtering#223

Merged
eledhwen merged 1 commit intomasterfrom
fix/wrapper-error-handling
Mar 17, 2026
Merged

fix(resilience4j): remove intermediate error wrapping clashing with resilience4j filtering#223
eledhwen merged 1 commit intomasterfrom
fix/wrapper-error-handling

Conversation

@eledhwen
Copy link
Collaborator

No description provided.

@eledhwen eledhwen requested review from STMW237 and theo-malka March 16, 2026 14:43
@eledhwen eledhwen self-assigned this Mar 16, 2026
@eledhwen eledhwen added the bug Something isn't working label Mar 16, 2026
@codecov-commenter
Copy link

codecov-commenter commented Mar 16, 2026

Codecov Report

❌ Patch coverage is 57.14286% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.98%. Comparing base (8bea08f) to head (bfb8c31).

Files with missing lines Patch % Lines
...ink/wrapper/circuitbreaker/CircuitBreakerSink.java 0.00% 3 Missing ⚠️
...tep/wrapper/circuitbreaker/CircuitBreakerStep.java 0.00% 2 Missing ⚠️
...ine/resilience4j/sink/wrapper/retry/RetrySink.java 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #223      +/-   ##
============================================
+ Coverage     80.79%   81.98%   +1.19%     
+ Complexity      876      873       -3     
============================================
  Files           166      166              
  Lines          2452     2420      -32     
  Branches        134      134              
============================================
+ Hits           1981     1984       +3     
+ Misses          388      353      -35     
  Partials         83       83              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@eledhwen eledhwen force-pushed the fix/wrapper-error-handling branch from 1f12546 to bfb8c31 Compare March 16, 2026 14:51
@eledhwen eledhwen merged commit a679c38 into master Mar 17, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants