Review: Approve +1 here though I note that we also don't mock os.path.exists in this unittest when looking at stop paths. curtin/block/clear_holders.py. line:279
Not sure if we really need to mock that or not -- https://code.launchpad.net/~daniel-thewatkins/curtin/+git/curtin/+merge/382770 Your team curtin developers is subscribed to branch curtin:master. -- Mailing list: https://launchpad.net/~curtin-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~curtin-dev More help : https://help.launchpad.net/ListHelp
