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.

Sign up for GitHub

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

Jump to bottom

use yaml.safe_load instead of load #105

Closed
wants to merge 1 commit into from
Closed

use yaml.safe_load instead of load #105

wants to merge 1 commit into from

Conversation

jarondl
Copy link

@jarondl jarondl commented Aug 10, 2017

Fix #103 by replacing all calls to load by safe_load. This exists
for both ruamel and PyYAML.

@jarondl
use yaml.safe_load instead of load
41c2d5a 
Fix Grokzen#103 by replacing all calls to `load` by `safe_load`. This exists
for both ruamel and PyYAML.
@coveralls
Copy link

coveralls commented Aug 10, 2017 •
edited

Coverage Status

Coverage remained the same at 87.2% when pulling 41c2d5a on jarondl:yaml-safeload into e7fac6a on Grokzen:unstable.

@Grokzen Grokzen closed this Dec 3, 2017
@Grokzen
Copy link
Owner

Grokzen commented Feb 6, 2018

Manually merged it in commit 74afb81

@Grokzen Grokzen mentioned this pull request Feb 6, 2018
Closed
@Grokzen Grokzen added the Merged label Feb 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jarondl @coveralls @Grokzen