kapitan
kapitan copied to clipboard
Add --tree-style-output compile option
Fixes issue #679
Proposed Changes
Add an option to not create folders with target names and instead merge output_paths of all targets in one tree.
Overall looks reasonable, but tests expect a different directory structure I believe so they should also be adjusted/fixed before merging.
Hey @roman8422 thanks for this - can you add a few test cases?
Hey @roman8422 thanks for this - can you add a few test cases?
Sure. I've had some personal stuff to deal with lately. Will fix this PR in the next couple of weeks. Thanks for willingness to accept it.
Hey @roman8422 any luck with this?
@ramaro I pushed my latest code change but I can't make unit tests work on master branch before I even start writing tests for my code. Looks like it is related to this issue https://github.com/kapicorp/kapitan/issues/358. Any advice how to overcome this?
Thanks @roman8422 ! No problem, this is a known issue which we're fixing with the latest merges. I think this is good to go so merging. Thanks again
@roman8422 you should be good to proceed now!
@ramaro "Docker Build and Push" action is constantly failing. Is there something I can do to fix that?
@roman8422 thanks for the patience. This looks good, thanks! Merging.
@ramaro I noticed that this PR wasn't merged and pushed a minor typo fix. Any plans to merge it any time soon? Edit: I see there are merge conflicts. Will resolve them and push new change.
@roman8422 apologies, I meant to merge this for v0.30 for missed it. Merging now so it will be official in v0.31. Thank you!
@roman8422 actually no, sorry - spoke too soon as tests don't pass