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

PyYaml > 6 requires loader argument #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

PyYaml > 6 requires loader argument #1

wants to merge 1 commit into from

Conversation

jack89roberts
Copy link

Just did this for the Turing fork of the material. Replace yaml.load/dump with safe_load / safe_dump. See yaml/pyyaml#576.

Replace with safe_load / safe_dump as used earlier in the material

yaml/pyyaml#576
@jack89roberts
Copy link
Author

Note we have a reworked version of this repo here for the Turing version of the course:
https://github.com/alan-turing-institute/bad-boids

The main difference is that the original version here requires the contents of fixture.yml to be changed for the tests on the better_boids branch to pass, which students found confusing as we talk about refactoring changing the style of the code but not the results. Also adds some niceties like a black and flake8 actions workflow.

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.

1 participant