carvel-dev / vendir

Easy way to vendor portions of git repos, github releases, helm charts, docker image contents, etc. declaratively
https://carvel.dev/vendir
Apache License 2.0
269 stars 47 forks source link

ensure tags are fetched if a ref is present #264

Closed Laevos closed 1 year ago

Laevos commented 1 year ago

Fixes #263

neil-hickey commented 1 year ago

So this particular fix just reverts the feature we implemented. So we will submit up a fix that solves for the feature working, and keep it's intended behaviour. Thanks @Laevos

Laevos commented 1 year ago

sgtm, thanks folks!

Jun 8, 2023 14:20:31 Neil Hickey @.***>:

So this particular fix just reverts the feature we implemented. So we will submit up a fix that solves for the feature working, and keep it's intended behaviour. Thanks @Laevoshttps://github.com/Laevos

— Reply to this email directly, view it on GitHubhttps://github.com/carvel-dev/vendir/pull/264#issuecomment-1583358861, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABKQRTERXXVOSB6LBVISZPTXKI6Z3ANCNFSM6AAAAAAY7W5ULM. You are receiving this because you were mentioned.Message ID: @.***>