923de4506e
feat: ApplicationModule with tentative architecture and placeholder implementation of first few features
55143dcad4
Merge pull request 'feat: add dry-run functionality and similar dependency' (#62) from feat/dryRun into master
That is inconvenient to use. Should be much shorter with something like :
I guess operator is not any string? Probably should be an enum too.
I think we already do have a Label type somewhere. I think it would be more appropriate here than String. That's true for all the label related fields in this file.
Not a String, should be a path type of some sort. I think we already handle this somewhere else correctly.
Yeah ok, I see what you did there.
But this is not a good use case for Macros. Macros are a last resort when the type semantics are already correct and we can't rely on the regular rust type…
OK that makes sense. Why not implement Display? I guess the format is not the expected one?
Also this file really deserves some tests to make sure the config output is correct.