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

Deprecatd config.save() in favor of OmegaConf.save() #66

Closed
omry opened this issue Nov 8, 2019 · 0 comments · Fixed by #68
Closed

Deprecatd config.save() in favor of OmegaConf.save() #66

omry opened this issue Nov 8, 2019 · 0 comments · Fixed by #68
Milestone

Comments

@omry
Copy link
Owner

omry commented Nov 8, 2019

Since Config is a dynamic data type, I want to minimize the functions that exists on it to the bare minimum.
A user can easily create a save field that would potentially conflict with the save() function.
A future version would remove config.save() completely.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant