Skip to content

Conversation

@Cardosaum
Copy link
Contributor

@Cardosaum Cardosaum commented Apr 11, 2025

Leverage existing standard library method to simplify implementation. No changes to public API or MSRV.

@ti-chi-bot
Copy link

ti-chi-bot bot commented Apr 11, 2025

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

Leverage existing standard library method to simplify implementation. No changes to public API or MSRV.

Signed-off-by: Matheus <[email protected]>
@Cardosaum Cardosaum force-pushed the cardosaum/simplify-duration-to-seconds branch from 0b65dde to 7a8e057 Compare April 11, 2025 16:52
@lucab
Copy link
Member

lucab commented Apr 15, 2025

@Cardosaum Thanks for the patch!
I'm happy to land something like this, but I think that it can be simplified even further while we are it.

After removing the aforementioned method in a previous commit, this test became obsolete.

Signed-off-by: Matheus <[email protected]>
@Cardosaum Cardosaum requested a review from lucab April 15, 2025 17:31
@Cardosaum
Copy link
Contributor Author

Thank you for the review @lucab!
Addressed the changed requested, please let me know if there's anything else missing :)

@lucab lucab merged commit 1d3174b into tikv:master Apr 16, 2025
8 checks passed
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