Skip to content

use safe_load because of yaml.load deprecation#75

Open
jpmens wants to merge 1 commit intoearl:masterfrom
jpmens:safe_load
Open

use safe_load because of yaml.load deprecation#75
jpmens wants to merge 1 commit intoearl:masterfrom
jpmens:safe_load

Conversation

@jpmens
Copy link

@jpmens jpmens commented Mar 20, 2019

PyYAML's yaml.load has been deprecated. This patch addresses that by using safe_load() in place of load().

Addresses #74

@jpmens
Copy link
Author

jpmens commented Mar 21, 2019

Submitted this PR at seveas#1

@sandi-go
Copy link

When can we expect the PR to be merged? I checked the Travis CI and found that it failed in installing setup_tools and I don't think there is any issue with the PR itself.

@ysmolski
Copy link

This is rather important fix. PR itself it fine, but this repo looks abandoned by the author. As maintainer of beanstalkd I ask the author of this repo to think about transferring this client to the beastalkd org, so more people can keep maintaining it.

https://github.com/beanstalkd/

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.

3 participants