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

Avoid YAML deprecation #4603

Merged
merged 2 commits into from Mar 16, 2019
Merged

Avoid YAML deprecation #4603

merged 2 commits into from Mar 16, 2019

Conversation

@mrocklin
Copy link
Member

@mrocklin mrocklin commented Mar 16, 2019

Fixes #4597

https://github.com/yaml/pyyaml/wiki/PyYAML-yaml.load(input)-Deprecation

  • Tests added / passed
  • Passes flake8 dask
@mrocklin
Copy link
Member Author

@mrocklin mrocklin commented Mar 16, 2019

Once this is in and released I suggest that we point all other repositories towards dask.config.yaml_load

@mrocklin
Copy link
Member Author

@mrocklin mrocklin commented Mar 16, 2019

Merging in 24 hours if there are no comments

dask/config.py Outdated Show resolved Hide resolved
@mrocklin
Copy link
Member Author

@mrocklin mrocklin commented Mar 16, 2019

@jakirkham jakirkham merged commit caa2da0 into dask:master Mar 16, 2019
2 checks passed
@jakirkham
Copy link
Member

@jakirkham jakirkham commented Mar 16, 2019

Thanks Matt :)

@jcrist jcrist deleted the yaml-loads branch Apr 19, 2019
jorge-pessoa pushed a commit to jorge-pessoa/dask that referenced this issue May 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants