Skip to content

Conversation

@codelion
Copy link
Contributor

@codelion codelion commented May 8, 2024

When the default yaml file has no values, it would make inputs, None this will default to empty dict in that case.

@codelion codelion requested a review from CTY-git May 8, 2024 11:48
@patched-admin
Copy link
Contributor

This code change involves assigning a default empty dictionary to the variable inputs if the yaml.safe_load() function is unable to load any data from the specified config or patchwork config file. This ensures that inputs will always have a value, even if the file reading operation fails.


Copy link
Contributor

@CTY-git CTY-git left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@CTY-git CTY-git merged commit 6c1ecc7 into main May 8, 2024
@codelion codelion deleted the fix-default-config-none-error branch May 9, 2024 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants