feat(condition): added panic recovery on condition by fredbi · Pull Request #95 · go-openapi/testify (original) (raw)
Conversation
Extended semantics for conditions used in async.
A panicking condition is recovered and treated as an error (marked with a sentinel error).
Updated documentation.
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.
Extended semantics for conditions used in async. A panicking condition is recovered and treated as an error (marked with a sentinel error).
Updated documentation.
Signed-off-by: Frederic BIDON fredbi@yahoo.com
Codecov Report
✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.34%. Comparing base (7089d2c) to head (3a7532e).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.
Additional details and impacted files
@@ Coverage Diff @@ ## master #95 +/- ##
Coverage 91.34% 91.34%
Files 94 94
Lines 12114 12125 +11
- Hits 11065 11076 +11
Misses 834 834
Partials 215 215
☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.
fredbi deleted the feat/async-panic-guard branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
[ Show hidden characters]({{ revealButtonHref }})