Skip to content
This repository has been archived by the owner on Nov 6, 2019. It is now read-only.

Change the registry api to return a tag name #77

Merged
merged 1 commit into from
Jul 10, 2018

Conversation

jbowes
Copy link
Member

@jbowes jbowes commented Jul 10, 2018

This opens things up for us to select a tag name that doesn't match the
github release name.

This opens things up for us to select a tag name that doesn't match the
github release name.
@codecov
Copy link

codecov bot commented Jul 10, 2018

Codecov Report

Merging #77 into master will increase coverage by 0.15%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #77      +/-   ##
==========================================
+ Coverage   18.96%   19.12%   +0.15%     
==========================================
  Files          22       23       +1     
  Lines        2452     2458       +6     
==========================================
+ Hits          465      470       +5     
- Misses       1956     1957       +1     
  Partials       31       31
Impacted Files Coverage Δ
pkg/imagepolicy/controller.go 12.59% <50%> (ø) ⬆️
pkg/registry/registry.go 83.33% <83.33%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a497d8a...410cf73. Read the comment docs.

// no matching tag being found.
func IsTagNotFoundError(err error) bool {
_, ok := err.(tagNotFoundError)
return ok
Copy link
Contributor

Choose a reason for hiding this comment

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

I like these type of errors 🙌

@jbowes jbowes merged commit daf3378 into master Jul 10, 2018
@jbowes jbowes deleted the what-registry-tag-matches branch July 10, 2018 23:38
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.

None yet

2 participants