-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
chore: backport #11587 #11777
chore: backport #11587 #11777
Conversation
@robert-zaremba why is the diff here so different from the original #11646 PR? |
@alexanderbez I forgot to change the target for this PR (was set automatically to main rather than the release branch). Updated. |
@robert-zaremba I think CI is stuck for some reason. Mind forcing a commit to jump start it? |
@alexanderbez some "artificial" commit? |
* feature: Clone and Merge methods on TraceContext * chore: use TraceContext.Merge method in SetTracingContext * feature: add Store name in TraceContext during execution * chore: update rootmulti Store TraceContext on *Store after Merge()
8469a49
to
d086df7
Compare
https://stackoverflow.com/questions/52200096/github-pull-request-waiting-for-status-to-be-reported But it looks like it's running now anyway. |
Backporting #11587
This feature is not breaking but requested by few teams.
We can't do it with mergifyio because we renamed the branch (see comments in #11646 (comment))
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
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