Rebased and removed WIP tag, should be good merge. Thanks for the speedy review!
Wow, that was fast! Thanks so much for the quick change.
BTW, not sure if it was related to branch name changes, but noticed the commit log messages for podman-in-podman issues. It was a web…
Updated PR based on feedback, things are looking cleaner now thanks to your suggestions. 😄
Verified again that test fails with original check_all_source_directories_exist
function and…
Will switch from integ to unit test, which will definitely be simpler and more pointed. I was trying to copy the style of the only test I saw using source_directories_must_exist
, without reading…
Fair enough! Will switch to list comprehension to better fit existing code style. Agree it is more readable, which is the common case so makes sense to optimize style for it.