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

2.0-rc1: YAMLLoadWarning #584

Closed
wren opened this issue Apr 10, 2019 · 2 comments
Closed

2.0-rc1: YAMLLoadWarning #584

wren opened this issue Apr 10, 2019 · 2 comments
Labels
🔒 Outdated

Comments

@wren
Copy link
Member

wren commented Apr 10, 2019

First of all, thanks for the code! This project really has a lot of promise.

I was exploring the 2.0-rc1 branch, and I get this warning when doing any command (including jnrl --version).

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

It looks like there are a few calls that use the older style of calling yaml.load.
maebert pushed a commit that referenced this issue Apr 14, 2019
@wren @maebert
Fix issue #584 YAMLLoadWarning (#585)
bcfd2d4
@maebert
Copy link
Contributor

maebert commented Apr 14, 2019

Thanks for the PR!

@maebert maebert closed this as completed Apr 14, 2019
wren added a commit that referenced this issue Apr 18, 2020
@wren
Fix issue #584 YAMLLoadWarning (#585)
13631b9
@lock
Copy link

lock bot commented May 21, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot added the 🔒 Outdated label May 21, 2020
@lock lock bot locked as resolved and limited conversation to collaborators May 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
🔒 Outdated
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@maebert @wren