Skip to content
This repository was archived by the owner on Oct 19, 2024. It is now read-only.

Distributed tracing#206

Merged
serras merged 9 commits intomasterfrom
distributed-tracing
Jul 20, 2020
Merged

Distributed tracing#206
serras merged 9 commits intomasterfrom
distributed-tracing

Conversation

@serras
Copy link
Copy Markdown
Collaborator

@serras serras commented Jun 22, 2020

Blocked until mtth/tracing#2 is merged

@serras serras force-pushed the distributed-tracing branch from c2cf4d5 to 8407448 Compare June 22, 2020 12:45
@serras serras mentioned this pull request Jun 22, 2020
Comment thread instrumentation/tracing/src/Mu/Instrumentation/Tracing.hs
@cb372
Copy link
Copy Markdown
Member

cb372 commented Jun 23, 2020

LGTM

@serras
Copy link
Copy Markdown
Collaborator Author

serras commented Jun 23, 2020

I am hesitant to merge until a new version of tracing supporting MonadBaseControl comes out.

@serras serras force-pushed the distributed-tracing branch from fbe1dc6 to 5c35d3f Compare July 3, 2020 15:10
@serras serras marked this pull request as ready for review July 14, 2020 09:35
@serras serras requested a review from kutyel as a code owner July 14, 2020 09:35
@serras
Copy link
Copy Markdown
Collaborator Author

serras commented Jul 14, 2020

Updated to refer to the forked version of the tracing library

@serras serras force-pushed the distributed-tracing branch from 63b3a4f to 4221694 Compare July 20, 2020 08:42
@serras serras requested a review from cb372 July 20, 2020 08:42
Copy link
Copy Markdown
Member

@kutyel kutyel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@serras serras merged commit 09820af into master Jul 20, 2020
@kutyel kutyel deleted the distributed-tracing branch July 20, 2020 10:34
kutyel pushed a commit that referenced this pull request Jan 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants