Skip to content

Implement panic recovery for resilience (issue-17)#19

Merged
cinar merged 1 commit intomainfrom
feature/panic-recovery
Mar 6, 2026
Merged

Implement panic recovery for resilience (issue-17)#19
cinar merged 1 commit intomainfrom
feature/panic-recovery

Conversation

@cinar
Copy link
Copy Markdown
Owner

@cinar cinar commented Mar 6, 2026

Pull Request

Description

  • Added WithPanicRecovery functional option to enable panic handling.
  • Introduced PanicError to encapsulate recovered panic values and stack traces.
  • Modified execution logic to treat recovered panics as retryable errors.
  • Added comprehensive unit tests for sequential and hedged execution.

Fixes #17

- Added WithPanicRecovery functional option to enable panic handling.
- Introduced PanicError to encapsulate recovered panic values and stack traces.
- Modified execution logic to treat recovered panics as retryable errors.
- Added comprehensive unit tests for sequential and hedged execution.
@cinar cinar merged commit b32a23d into main Mar 6, 2026
4 checks passed
@cinar cinar deleted the feature/panic-recovery branch March 6, 2026 04:57
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 75.00000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.61%. Comparing base (075df51) to head (e797161).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
retry.go 69.23% 3 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #19      +/-   ##
==========================================
- Coverage   93.30%   92.61%   -0.69%     
==========================================
  Files           8        8              
  Lines         433      447      +14     
==========================================
+ Hits          404      414      +10     
- Misses         21       24       +3     
- Partials        8        9       +1     

☔ 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.

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.

Feature: WithPanicRecovery option for "Let It Crash" philosophy

2 participants