gocardless / statesman

A statesmanlike state machine library.
https://gocardless.com/blog/statesman/
MIT License
1.78k stars 163 forks source link

Add SemVer compatibility badge to README #323

Closed greysteil closed 6 years ago

greysteil commented 6 years ago

Would you folks be up for showing your SemVer compatibility in a badge on your readme? We've just added the ability to do that using data from the test-runs of all the pull requests Dependabot creates - what do you reckon?

Here's what the badge will look like, and where it will link to:

SemVer

Any feedback super appreciated!

greysteil commented 6 years ago

@nickcampbell18 - any thoughts?

greysteil commented 6 years ago

Updated the text of the badge to semver stability, which should make the use case clearer. Any thoughts?

nickcampbell18 commented 6 years ago

No strong feelings. @hmac, thoughts?

hmac commented 6 years ago

Yeah, why not? I assume the number is rounded to the nearest 10%?

greysteil commented 6 years ago

It’s exact - just happens that statesman is at a round number at the moment

greysteil commented 6 years ago

@nickcampbell18 - updated!

hmac commented 6 years ago

Dependabot has updated statesman between SemVer compatible versions 39 times across 12 projects so far. 90% of those updates passed CI.

So 35.1 CI passes?

greysteil commented 6 years ago

OK, so it's rounded to the nearest 1%. You know what I mean @hmac 😜

petehamilton commented 6 years ago

@nickcampbell18 @hmac any idea what's going on with CI? They're all just pending but I can't find anything in CircleCI πŸ€”

This isn't going to break tests, so I'm just going to merge this in 🚒