Skip to content
This repository was archived by the owner on Feb 25, 2020. It is now read-only.

Add the images being pushed#3

Open
timfallmk wants to merge 1 commit intoneonadventures:masterfrom
timfallmk:images-pushed
Open

Add the images being pushed#3
timfallmk wants to merge 1 commit intoneonadventures:masterfrom
timfallmk:images-pushed

Conversation

@timfallmk
Copy link
Copy Markdown

Just a quick change to include the image tag being built. If a repository has multiple tags in an automated build, they'll all show up as the same notification.

@andykent
Copy link
Copy Markdown
Contributor

Hmm, I get the issue but I do think the formatting could be a bit nicer, maybe we should truncate the SHAs to 6 chars, git style? What do you think?

@timfallmk
Copy link
Copy Markdown
Author

Yeah, I was hoping they would expose the tags the images had instead of the hashes. Truncated seems like it would work.

@timfallmk
Copy link
Copy Markdown
Author

@andykent Have any magic up your sleeve for doing the truncation on the same line? ;)

@n0mer
Copy link
Copy Markdown

n0mer commented Aug 11, 2015

@timfallmk , is it possible to include label ("tag") of pushed image?
And, https://docs.docker.com/docker-hub/builds/#webhooks page says that it is also possible to include "builder" (author) information.

@timfallmk
Copy link
Copy Markdown
Author

Just a ping on this. Should be possible to add any of those fields linked to above.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants