-
Notifications
You must be signed in to change notification settings - Fork 147
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
fix: rework examples #2646
fix: rework examples #2646
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2646 +/- ##
==========================================
+ Coverage 84.81% 84.85% +0.04%
==========================================
Files 284 284
Lines 71281 71281
==========================================
+ Hits 60457 60487 +30
+ Misses 10824 10794 -30 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
Signed-off-by: Ivaylo Nikolov <[email protected]>
376b9db
to
5cccb97
Compare
Quality Gate failedFailed conditions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description:
Uncomments non-working example after the latest changes and applies necessary changes for them to work again.
Fixes #2619
Notes for reviewer:
Removed the last steps in the example(where we add the signatures) because after the latest sig map changes adding back the signatures after using
removeSignature
functionalities create unnecessary code gymnastics.Checklist