Skip to content
This repository was archived by the owner on Jul 2, 2021. It is now read-only.

run: don't fail if object missing from db#105

Closed
powerjg wants to merge 1 commit intomasterfrom
fix-run-exception
Closed

run: don't fail if object missing from db#105
powerjg wants to merge 1 commit intomasterfrom
fix-run-exception

Conversation

@powerjg
Copy link
Member

@powerjg powerjg commented Aug 10, 2020

In loadFromDict, if there is an artifact that's not found in the
database, the current behavior is to raise an exception. This change
instead inserts a "placeholder" dictionary with the original UUID and a
note that the data can't be found. Currently, this function is only used
when extracting runs from the database.

Signed-off-by: Jason Lowe-Power jason@lowepower.com

In loadFromDict, if there is an artifact that's not found in the
database, the current behavior is to raise an exception. This change
instead inserts a "placeholder" dictionary with the original UUID and a
note that the data can't be found. Currently, this function is only used
when extracting runs from the database.

Signed-off-by: Jason Lowe-Power <jason@lowepower.com>
@powerjg powerjg requested a review from hnpl August 10, 2020 18:35
@powerjg
Copy link
Member Author

powerjg commented Aug 10, 2020

NVM, skip this one for now. mypy is complaining and I'm going to refactor

@powerjg
Copy link
Member Author

powerjg commented Aug 10, 2020

I took care of this problem in artifact instead: #103

@powerjg powerjg closed this Aug 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant