Skip to content
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

fix PyYAML warning on call of yaml.load(…) #186

Merged
merged 1 commit into from
Jul 28, 2019
Merged

fix PyYAML warning on call of yaml.load(…) #186

merged 1 commit into from
Jul 28, 2019

Conversation

agebhar1
Copy link
Contributor

Since PyYAML v5.1 a warning on deprecation is printed:

YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated,
as the default Loader is unsafe. Please read https://msg.pyyaml.org/load
for full details.

Use the FullLoaders suggared variant yaml.full_load according to the
documentation [1] if it's present. Then it's assumed that PyYAML >= v5.1 is
used.

[1] https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation#how-to-disable-the-warning

Since PyYAML v5.1 a warning on deprecation is printed:

> YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated,
> as the default Loader is unsafe. Please read https://msg.pyyaml.org/load
> for full details.

Use the `FullLoader`s suggared variant `yaml.full_load` according to the
documentation [1] if it's present. Then it's assumed that PyYAML >= v5.1 is
used.

[1] https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation#how-to-disable-the-warning
@coveralls
Copy link

Coverage Status

Coverage remained the same at ?% when pulling f8751ca on agebhar1:feature/PyYAML5 into 4aa6fe9 on beerfactory:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at ?% when pulling f8751ca on agebhar1:feature/PyYAML5 into 4aa6fe9 on beerfactory:master.

@njouanin njouanin merged commit 4d44a05 into njouanin:master Jul 28, 2019
@agebhar1 agebhar1 deleted the feature/PyYAML5 branch July 28, 2019 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants