Skip to content

Conversation

@febo
Copy link
Collaborator

@febo febo commented Dec 4, 2025

Problem

Few bits on the documentation could be more clear or use better formatting.

Solution

Aded small tweaks to improve presentation and clarity.

README.md Outdated
<a href="https://github.com/anza-xyz/pinocchio/actions/workflows/main.yml"><img src="https://img.shields.io/github/actions/workflow/status/anza-xyz/pinocchio/main.yml?logo=GitHub" /></a>
<a href="https://crates.io/crates/pinocchio"><img src="https://img.shields.io/crates/v/pinocchio?logo=rust" /></a>
<a href="https://docs.rs/pinocchio"><img src="https://img.shields.io/docsrs/pinocchio?logo=docsdotrs" /></a>
<a href="https://docs.rs/pinocchio"><img src="https://img.shields.io/badge/docs-latest-brightgreen?logo=docsdotrs" /></a>
Copy link

Choose a reason for hiding this comment

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

This is hardcoding the status as green though right? We kinda lose the dynamic nature of the badge if so.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good point, reverted the change here.

Copy link
Collaborator

@joncinque joncinque left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Copy link

@grod220 grod220 left a comment

Choose a reason for hiding this comment

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

LGTM

@febo febo merged commit bb90c38 into main Dec 4, 2025
8 checks passed
@febo febo deleted the febo/docs-tweaks branch December 4, 2025 11:18
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.

4 participants