salaboy
salaboy
@hasheddan hmm.. now that I am looking at this comment again.. what do you mean by `machinery` here? I didn't need much besides using (from the package/ dir) : ```...
@muvaf I am .. let me get it done tomorrow early morning.
@muvaf is there something else needed for this PR to be merged?
@jbw976 I think it makes a lot of sense to not include it in the code itself, as that is not needed there for the provider final version running inside...
I am happy to drive and bring people together around this topic to create a charter. I am sure that @ThomasVitale is happy to join too.
Thank you all for commenting out here.. I will work a bit on the files created by @Riaankl , if you want to **add comments** or use **suggestion mode** that...
@Titof974 please **use suggestion mode** or **add comments**.. if not it gets really hard for me to review and understand changes
@danieloh30 that is correct.. check the PR content: https://github.com/cncf/tag-app-delivery/pull/638/files
@ThomasVitale thanks a lot for that! I was waiting to check the rendering
@danieloh30 @ThomasVitale I tried to make the diagrams a bit better.. I think that they are a bit clearer now.. but there is a lot of room for improvements