Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Reproducible Builds badge to README #333

Merged
merged 2 commits into from
Nov 3, 2022

Conversation

hboutemy
Copy link
Contributor

@hboutemy hboutemy commented Nov 3, 2022

congratulations, your latest releases were proven reproducible
Let's celebrate with this badge :)

Copy link
Member

@Stephan202 Stephan202 left a comment

Choose a reason for hiding this comment

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

Thanks! Looks very nice! 🎉

image

Made one suggestion around link naming :)

Suggested commit message:

Add Reproducible Builds badge to README (#333)

NB: I didn't forget about jvm-repo-rebuild/reproducible-central#93!

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@@ -226,3 +227,5 @@ guidelines][contributing].
[refaster]: https://errorprone.info/docs/refaster
[refaster-rules-bigdecimal]: https://github.com/PicnicSupermarket/error-prone-support/blob/master/error-prone-contrib/src/main/java/tech/picnic/errorprone/refasterrules/BigDecimalRules.java
[refaster-rules]: https://github.com/PicnicSupermarket/error-prone-support/blob/master/error-prone-contrib/src/main/java/tech/picnic/errorprone/refasterrules/
[reproducible-builds-badge]: https://img.shields.io/badge/Reproducible_Builds-ok-success?labelColor=1e5b96
Copy link
Member

Choose a reason for hiding this comment

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

How was the color selected? 😄

(I guess that eventually it'd be cool to have dynamic badges that go red if the latest release is no longer reproducible ✨.)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

color was chosen after Reproducible Builds logo

@Stephan202 Stephan202 added this to the 0.6.0 milestone Nov 3, 2022
@rickie rickie added the documentation A documentation update label Nov 3, 2022
@rickie rickie self-requested a review November 3, 2022 19:07
Copy link
Member

@rickie rickie left a comment

Choose a reason for hiding this comment

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

Nice contribution @hboutemy! 🚀

Changes, @Stephan202's suggestion, and the suggested commit message LGTM!

Let's see whether this works on remote forks 👀
@Stephan202 Stephan202 merged commit 7febccb into PicnicSupermarket:master Nov 3, 2022
@rickie rickie changed the title add Reproducible Builds badge Add Reproducible Builds badge to README Nov 3, 2022
@hboutemy hboutemy deleted the patch-1 branch November 3, 2022 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation A documentation update
Development

Successfully merging this pull request may close these issues.

3 participants