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

PR for Docker Hub Automated Builds #661

Merged
merged 2 commits into from
Jul 25, 2016
Merged

Conversation

jrottenberg
Copy link

#241

Query https://registry.hub.docker.com/v2/repositories/USER/IMAGE/ to get the boolean 'is_automated'

'automated build' for the left part, 'docker automated' or 'docker automated build' didn't sound as good.

@jrottenberg
Copy link
Author

Maybe 'docker build' on the left, with 'automated' or 'manual' on the right would work better. Let me know.

@espadrine espadrine changed the title PR for badges/shields#241 PR for Docker Hub Automated Builds Mar 29, 2016
@espadrine
Copy link
Member

Thanks for your contribution!

Maybe 'docker build' on the left, with 'automated' or 'manual' on the right would work better.

You are absolutely right! It would be lovely.

@jrottenberg
Copy link
Author

Great, let me know if it's merge-able now.

@tclarke
Copy link

tclarke commented Apr 5, 2016

Any eta on merge and deploy for this PR? Can't wait to use it!

@hairyhenderson
Copy link

Nice! Would be good to see this in - is there anything blocking this getting merged, @espadrine?

@seblegall
Copy link

Hi,

Any news of this PR? I will be glad to be one of the first to use this badge :-)

@jakirkham
Copy link

Are we sure this is working? I can't seem to see the badges when previewing the HTML.

@espadrine espadrine merged commit e728f23 into badges:master Jul 25, 2016
@espadrine
Copy link
Member

Thanks for the contribution!

Unfortunately, I am not in the greatest situation to push to production. I hope to push this change live this week-end.

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.

6 participants