Skip to content

docs: enrich macro docs with more details/examples #542

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rupert648
Copy link

Was doing some work with these macros and had a look and decided the docs could use a bit of love. Tried to add more examples and references to other items. Let me know what you think!

Copy link

ti-chi-bot bot commented Mar 14, 2025

Welcome @rupert648! It looks like this is your first PR to tikv/rust-prometheus 🎉

///
/// This macro can be used in three ways:
///
/// ```
Copy link
Member

Choose a reason for hiding this comment

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

Here and below, it looks like some doc-tests are failing due to code mismatches (some missing use directives and some wrong API calls).
Can you run a cargo test and fix all the relevant issues?

@lucab
Copy link
Member

lucab commented Mar 19, 2025

Thanks for the additional docs!
CI is currently red here, there are some doc-tests which need to be fixed.
It looks like the DCO bot expects a sign-off from Rupert Carr <[email protected]>, can you maybe update your commit?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants