-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: add sdk.LogDeferred
to report errors in defers
#13619
refactor: add sdk.LogDeferred
to report errors in defers
#13619
Conversation
There's technically nothing about |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #13619 +/- ##
==========================================
+ Coverage 54.81% 54.83% +0.02%
==========================================
Files 647 647
Lines 55797 55794 -3
==========================================
+ Hits 30586 30597 +11
+ Misses 22720 22705 -15
- Partials 2491 2492 +1
|
+1 This helper just logs the error of any given function. Maybe a clearer name would be good, |
sdk.LogDeferred
to report errors in defers
Description
Closes: #11785
A new helper function called
LogDeferred
was added. This function it's used to report errors indefer
section. It was added indefer
statements ofx/bank
folder.Author Checklist
I have...
!
to the type prefix if API or client breaking changeCHANGELOG.md
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...
!
in the type prefix if API or client breaking change