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 deps badge to README #35

Merged
merged 1 commit into from
May 29, 2016
Merged

Add deps badge to README #35

merged 1 commit into from
May 29, 2016

Conversation

rrrene
Copy link
Contributor

@rrrene rrrene commented May 28, 2016

Hi Isaac,

we don't know each other, but I'd like to pitch you a new Elixir related badge:

Deps Status

It shows you an analysis for your dependencies. Behind the badge works a CI service written in Elixir which can notify you whenever important updates for your Hex packages are released.

This is still in its infancy, and I want to basically invite you to join the beta to test-drive this. I really believe that we can create a great service for the community with this. That said, don't feel any obligation to accept the PR. You can't hurt my feelings by voicing your honest opinion about this idea!

@whitfin
Copy link
Owner

whitfin commented May 29, 2016

@rrrene actually I filed an issue on Credo the other day ;)

I'll accept this sure - what exactly should the badge look like when everything is perfect? A solid green bar?

Thanks

@rrrene
Copy link
Contributor Author

rrrene commented May 29, 2016

Ehm, that's why I never lament the short attention span of others 😅

In case of your badge, everything is fine. The different greens are for the Mix envs:

  • Dark green: prod deps
  • Light green: other deps (dev, test, et al.)

There is a help page for it, but re-reading it now I realize it could be improved as well: https://beta.hexfaktor.org/help/badge

I created an issue to remember doing that! 👍

@whitfin whitfin merged commit f9a5df6 into whitfin:master May 29, 2016
@whitfin
Copy link
Owner

whitfin commented May 29, 2016

@rrrene neat. I merged it in :) thank you :)

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.

None yet

2 participants