-
Notifications
You must be signed in to change notification settings - Fork 121
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deprecated call to yaml.load #288
Comments
The fix should be similar to the code in this PR: chaoss/grimoirelab-sortinghat#200 @sumitskj this could be good for you for micro-task 9, what do you think ? :) |
Sure, I will try to fix this |
great! thank you @sumitskj |
I have added the Loader param in yaml.load() #289 but it has failed to pass tests. Can you see this @jgbarah @valeriocos . |
Thank you for working on this issue @sumitskj .The tests fail due to a missing package. I'll make a local test and merge it |
Merged, thank you @sumitskj |
When launching
sirmordred
with any configuration, you see two warnings:I think the fix is easy: just call yaml.load() with a
Loader
parameter.I know this is not exactly a bug, since it produces the right output. But I'm labeling as bug because it is kindda confusing for the user, anyway...
The text was updated successfully, but these errors were encountered: