Calamari
Calamari copied to clipboard
fix Python updateprogress()
According to the documentation the message for updateprogress in Python scripts is optional. However, when a message is not provided the script fails because the default for message is None which is then passed to the inner encode() function that does not like None.
In this fix the default value for message is changed from None to "" (empty string). Further this PR provided necessary test fixtures to assert the correct behavior.
Ping, just some minor changes. Do I need to do something to move this forward.
Thanks for the PR @smlng. I've created a fresh branch and PR, as our build scripts are exposed within this repository we do not automatically build community pull requests and have to trigger them manually. Some new security measures internally have prevented us from manually triggering these, and I'm waiting for confirmation whether I can. In the meantime, I've branched off of your changes and created a fresh pull request. I'll take a look at the changes early next week and see if we can get them merged in. Thanks again for the contribution!