Either I don't understand it or it's an outdated comment, I think those types of comment explaining WHAT the code is doing are generally bad. Comments should explain WHY. If you need to express a WHAT, use a function.
Weird comment here again, does not relate directly to the code block. I think I'd rather not have it.
This function, though pretty simple, deserves a test. Could be a good place for a doctest. Not necessary though, just spilling my thoughts.
So much better. I think there are more Rust idiomatic ways to handle that with Slices or something but it's good enough :
Those 8 lines should be extracted to a utilty function in the test module such as "init_test_maestro() -> Maestro"
This is just so much better. And it shows in the UX. Usually a good UX requires good code and good code produces a good UX.
C'est mieux. mais je pense que ca merite encore un peu d'amour si c'est pas trop difficile pour le rendering d'un Map.
Ca serait…
I generally feel wrong about not using an existing tool to deploy helm packages. I feel like using kustomize would be more reasonable here but at the same time it's not that hard to develop the…
Je viens de l'essayer, semble pas pire, mais je pense que ca prend quelques trucs pour que le UX soit correct :