Skip to content
This repository was archived by the owner on Jun 25, 2020. It is now read-only.

Use yaml.safe_load#134

Merged
wjwwood merged 1 commit into
vcstools:masterfrom
cottsay:safe_load
Jul 10, 2019
Merged

Use yaml.safe_load#134
wjwwood merged 1 commit into
vcstools:masterfrom
cottsay:safe_load

Conversation

@cottsay
Copy link
Copy Markdown
Contributor

@cottsay cottsay commented Apr 9, 2019

Tests will fail until #133 is addressed.

@raghavakumar8
Copy link
Copy Markdown

Looking forward to having this and #133 merged! YAMLLoadWarnings are annoying.

@rhaschke
Copy link
Copy Markdown

Ping. @tfoote, @wjwwood Can this be merged?

@wjwwood wjwwood self-assigned this Jul 10, 2019
@wjwwood wjwwood added the bug label Jul 10, 2019
@wjwwood
Copy link
Copy Markdown
Contributor

wjwwood commented Jul 10, 2019

I merged #133 and restarted the CI.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 81.992% when pulling 5397d6f on cottsay:safe_load into 3a6a2d3 on vcstools:master.

@wjwwood wjwwood merged commit c5cabae into vcstools:master Jul 10, 2019
@cottsay cottsay deleted the safe_load branch June 3, 2020 18:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants