Conversation
|
Huh, this is pretty neat! My only concern here is that it's not entirely future-proof. It's very possible that we will eventually depend on sub-dependencies like It's not a big deal since those badges are kind of unnecessary for old releases anyway (since they reflect the current state, not the old one), but it does maybe look a bit wonky if there's just a fully broken badge on all old releases. Perhaps it's worth it though, I'm not entirely sure yet 🤔 |
|
Good point, I hadn't thought about that. One way around it might be to hide the badge on crates.io entirely. If I remember correctly, you could use specific HTML comments to exclude parts of the Readme from being rendered... |
|
Found it. So it adds a bit of complexity, but should "future-proof" it. |
|
Hmm that's for rustdoc, but in this case we want to hide it from the README shown on crates.io, so it probably wouldn't work there? We don't embed the README into our crate docs, that page is separate |
Im not particularly big on badges (the bevy one is kinda neat since its actually helpful) but you could have them reflect the actual status by pointing to the tag i guess? But would be additional maintenance burden for a pretty small benefit. |
Objective
Make it easily visible which Bevy version Avian currently supports.
Solution
Add a badge to the README which automatically queries the currently used Bevy version.
This is implemented with the "Dynamic TOML Badge" feature from shields.io.
It's used to query the current Bevy version from the Cargo.toml of Avian 2D (assuming the Bevy version is consistent across 2D and 3D).
Clicking on the Badge links you to the version table in the README.
Note
If the badge is desired, I can of course re-order it or change the labels to match your preference.
Testing
N/A
Showcase
RENDERED
