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

Opentelemetry initializer #531

Merged

Conversation

dinosath
Copy link
Contributor

I thought of adding an opentelemetry initializer in the locos extras. Where should i add documentation for this initializer? Because in order for this one to work we need to have to turn off the init_logger

@jondot
Copy link
Contributor

jondot commented Apr 12, 2024

@dinosath thanks for this! looks really great!
For now, you can include a README.md right next to your initializer folder (ie turn opentelemetry.rs into a folder with a mod.rs and README.md). We'll include links to all of the README.md's in the docs eventually.

@dinosath
Copy link
Contributor Author

Done

@jondot
Copy link
Contributor

jondot commented Apr 30, 2024

Looks good -- can you run fmt on the code?

@kaplanelad kaplanelad merged commit d0d5339 into loco-rs:master May 2, 2024
11 checks passed
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.

3 participants