-
-
Notifications
You must be signed in to change notification settings - Fork 306
feat: add martin-tracing-utils for configuring tracing (1/3)
#1747
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
base: main
Are you sure you want to change the base?
Conversation
for more information, see https://pre-commit.ci
|
@nyurik @sharkAndshark might it be possible to get an review on this one? 👉🏻👈🏻 This would really help me push towards #1720 which (for me) currently kind of blocks the refactoring #1805 (comment) |
|
We talked this out offliline and we will move this to different places:
I will reasses if this works and come back with a different PR |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
This is the first PR to break out of #1720 to make it reviewable.
The purpose is mainly to see if the added utilities are
martin-tracing-utilsokay.Current usage of this code would be