Just one more comment, I'll QA asap.
https://codereview.appspot.com/26740043/diff/20001/server/guiserver/bundles/base.py File server/guiserver/bundles/base.py (right):
https://codereview.appspot.com/26740043/diff/20001/server/guiserver/bundles/base.py#newcode188 server/guiserver/bundles/base.py:188: if bundle_id is not None and error is None: I the future has been cancelled then error is not defined here.
https://codereview.appspot.com/26740043/
« Back to merge proposal
Just one more comment, I'll QA asap.
https:/ /codereview. appspot. com/26740043/ diff/20001/ server/ guiserver/ bundles/ base.py guiserver/ bundles/ base.py (right):
File server/
https:/ /codereview. appspot. com/26740043/ diff/20001/ server/ guiserver/ bundles/ base.py# newcode188 guiserver/ bundles/ base.py: 188: if bundle_id is not None and error
server/
is None:
I the future has been cancelled then error is not defined here.
https:/ /codereview. appspot. com/26740043/