Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixes #289 Omegaconf chaging SafeLoader #295

Merged
merged 3 commits into from
Jul 12, 2020
Merged

Conversation

pereman2
Copy link
Contributor

Closes #289
By adding a class wrapper for yaml.SafeLoader it solves the problem.
Added test and news report.

@omry omry merged commit dda17b0 into omry:master Jul 12, 2020
@pereman2 pereman2 deleted the bug/safeloader3 branch July 12, 2020 22:15
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OmegaConf is changeing the pyyaml SafeLoader
2 participants