search

nais / docker-build-push

MIT License
0 stars 5 forks source link

ver. for action? #10

Closed ybelMekk closed 1 year ago

ybelMekk commented 1 year ago

Syns et action bør ha en pinned version, både for breaking changes ikke går direkt i main men også for at man skal vite hva man bruker i sitt action, hvis man bruker main så kan de vare hav som en går inn i main som man kjør i sitt action, så mulig vi må begynne å versionere dette action?

jhrv commented 1 year ago

I console-backend har vi gjort slik: https://github.com/nais/console-backend/blob/main/.github/workflows/main.yaml#L15

thokra-nav commented 1 year ago

Vi har versjon for actionet, v0. Det skal ikke være breaking changes i samme versjon, da må vi bumpe. (Vi har også v0.7.0 osv hvis man vil være mer spesifik) https://github.com/nais/docker-build-push/tags

Vil selv argumentere imot noe hardere pin enn på tag, spesielt for interne actions. For meg så er verdien i dette actionet at dersom man bruker det så vil det holdes oppdatert på en måte som forhindrer at buildet mitt stopper fordi NAIS f.eks. ønsker at man bruker et annet registry, etc.

Så usikker på hvem som har brukt @main, når alle eksempler viser @v0