Skip to content

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