Skip to content
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

mock: add README to tracing-mock #2357

Closed
wants to merge 2 commits into from
Closed

Conversation

hds
Copy link
Contributor

@hds hds commented Oct 25, 2022

Motivation

There has been interest around publishing tracing-mock to crates.io for some time. In order to make this possible, documentation and some code clean up is needed.

Specifically I want to have access to tracing-mock within parts of tokio to write tests that ensure that the correct calling location is picked up for all spawn* functions when the tracing feature is enabled.

Solution

This change starts that process by adding a README for tracing-mock. The README follows the standard format for all tracing crates and includes 2 examples. Additionally an integration test has been added for each example.

Refs: #539

There has been interest around publishing `tracing-mock` to crates.io
for some time. In order to make this possible, documentation and some
code clean up is needed.

This change starts that process by adding a README for `tracing-mock`.
The README follows the standard format for all `tracing` crates and
includes 2 examples. Additionally an integration test has been added for
each example.

Refs: #539
@hds hds requested review from hawkw and a team as code owners October 25, 2022 13:27
@davidbarsky
Copy link
Member

Thanks for opening this PR! Before I review the rest of this PR, can you retarget this PR to point towards master instead of v0.1.x? We'll handle backports.

@hds
Copy link
Contributor Author

hds commented Oct 28, 2022

Thanks for opening this PR! Before I review the rest of this PR, can you retarget this PR to point towards master instead of v0.1.x? We'll handle backports.

Sure, I'll do that. I was unsure about where to make these changes, so now I know. (-:

@hds
Copy link
Contributor Author

hds commented Nov 2, 2022

Closing this PR in favor of one targeting master instead: #2362

@hds hds closed this Nov 2, 2022
@hds hds deleted the hds/tracing-mock-readme-v0.1.x branch November 4, 2022 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants