Not sure about the handling of the credentials, default settings should be leaning a bit more towards high security and never delete I think.
lgtm, went quickly over it, I'll need to get a better feel once I start using it.
it would be better to resolve the version in the bash script here, at least it is not crappy inline yaml in ci config absolutely not reusable anywhere. But ideally it should happen in rust, harmony being an infrastructure tool it is relevant to have git understanding to resolve tags and other stuff.
This is about 5x more code than it should be, and 10x more documentation/comments. Do better.
We will have to make this simpler, we should not have to rebuild a new cli for every component of every app using harmony. This should be a macro call or a simple function call, I don't know yet but definitely not a 270 lines file.
This is making a simple thing hard and complicated. We already have an argo module, just use it. Don't rebuilt another one on top. Complete waste of lines of code and time and documentation.
should be done in rust, we don't want to depend on bash inlined in yaml it's not only locking-in and not reusable it is brittle.