-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat: add traces across relayer #205
Conversation
Go Test coverage is 63.6 %\ ✨ ✨ ✨ |
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.
Found deposit in core is missing deposit data and handler response.
Minor nitpick, but don't really like where ctx is renamed to ctxWithSpan. |
Go Test coverage is 63.6 %\ ✨ ✨ ✨ |
…actually finishing its execution
Go Test coverage is 63.5 %\ ✨ ✨ ✨ |
Go Test coverage is 63.5 %\ ✨ ✨ ✨ |
Recent changes require updates from ChainSafe/chainbridge-core#350 |
Go Test coverage is 64.6 %\ ✨ ✨ ✨ |
Closing temporarily. |
Description
Adds Traces across the whole repo.
This is highly depended on changes in CORE repo that needs to be released first
Related Issue Or Context
Closes: n/a
How Has This Been Tested? Testing details.
Types of changes
Checklist: