Skip to content

Wrap container exceptions in HandlerResolver - #330

Merged
vjik merged 2 commits into
masterfrom
improve-handler-resolver
Sep 5, 2026
Merged

Wrap container exceptions in HandlerResolver#330
vjik merged 2 commits into
masterfrom
improve-handler-resolver

Conversation

@vjik

@vjik vjik commented Sep 4, 2026

Copy link
Copy Markdown
Member
Q A
Is bugfix?
New feature?
Breaks BC?
Tests pass? ✔️

@codecov

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.53%. Comparing base (f178257) to head (8151701).

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #330      +/-   ##
============================================
+ Coverage     92.51%   92.53%   +0.01%     
- Complexity      381      382       +1     
============================================
  Files            59       59              
  Lines          1016     1018       +2     
============================================
+ Hits            940      942       +2     
  Misses           76       76              

☔ View full report in Codecov by Harness.
📢 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.

@vjik
vjik requested a review from a team September 4, 2026 15:15
@vjik vjik added the status:code review The pull request needs review. label Sep 4, 2026

@samdark samdark left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK but doesn't prevent consumer worker termination in case of error.

@vjik
vjik merged commit e11d8a0 into master Sep 5, 2026
28 of 29 checks passed
@vjik
vjik deleted the improve-handler-resolver branch September 5, 2026 04:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:code review The pull request needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants