-
Notifications
You must be signed in to change notification settings - Fork 10
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
LEDGER_ASSERT improvments and CX_ASSERT addition #504
Conversation
80d5665
to
6cae869
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #504 +/- ##
=======================================
Coverage 60.42% 60.42%
=======================================
Files 12 12
Lines 1660 1660
=======================================
Hits 1003 1003
Misses 657 657
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
b9f60d6
to
8616507
Compare
EDIT: not used anymore, but we never know, it might be useful at some point? For future, here is the script used to generate most of the PR (except includes and formatting):
|
36a4590
to
6ce2cab
Compare
6ce2cab
to
bb26bd8
Compare
bb26bd8
to
69c8890
Compare
Description
Add some documentation for
LEDGER_ASSERT
Allow
printf
like usage ofLEDGER_ASSERT
for error messageMinor cleanup on CX includes
Add
CX_ASSERT
macroChanges include