chainguard-images / .github

1 stars 10 forks source link

add sync labels #8

Closed cpanato closed 1 year ago

cpanato commented 1 year ago
cpanato commented 1 year ago

Thanks Carlos! This is probably a dumb idea, but could we skip the labels.json and just use the labels on this ,github repo to be the authoratative source?

i think we can, but then we will not have the audit part in case we need it (who knows 🤷‍♂️ ) and will make hard to know who change what/when

but i can implement that in a follow up if you would like/want

patflynn commented 1 year ago

Nah this is good. Thanks again!

On Thu, Nov 3, 2022 at 9:02 AM Carlos Tadeu Panato Junior < @.***> wrote:

Thanks Carlos! This is probably a dumb idea, but could we skip the labels.json and just use the labels on this ,github repo to be the authoratative source?

i think we can, but then we will not have the audit part in case we need it (who knows 🤷‍♂️ ) and will make hard to know who change what/when

but i can implement that in a follow up if you would like/want

— Reply to this email directly, view it on GitHub https://github.com/chainguard-images/.github/pull/8#issuecomment-1302069791, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB37SHOQ3BQBWFXZ5NKKDJDWGOZWBANCNFSM6AAAAAARVGQY3M . You are receiving this because your review was requested.Message ID: @.***>

cpanato commented 1 year ago

will merge to test this

patflynn commented 1 year ago

works towards #933