-
Notifications
You must be signed in to change notification settings - Fork 527
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
Core: Add dict functionality to OptionDict #2036
Conversation
Instead of selectively implementing some methods, you might want to consider the See here for an example. |
items()
and __getitem__
to OptionDictitems()
and __getitem__
to OptionDict
items()
and __getitem__
to OptionDictCo-authored-by: black-sliver <59490463+black-sliver@users.noreply.github.com>
* Options: Add support for `items()` and `__getitem__` to OptionDict * Options: have OptionDict inherit from Mapping * add typing to __getitem__ Co-authored-by: black-sliver <59490463+black-sliver@users.noreply.github.com> --------- Co-authored-by: black-sliver <59490463+black-sliver@users.noreply.github.com>
* Options: Add support for `items()` and `__getitem__` to OptionDict * Options: have OptionDict inherit from Mapping * add typing to __getitem__ Co-authored-by: black-sliver <59490463+black-sliver@users.noreply.github.com> --------- Co-authored-by: black-sliver <59490463+black-sliver@users.noreply.github.com>
* Options: Add support for `items()` and `__getitem__` to OptionDict * Options: have OptionDict inherit from Mapping * add typing to __getitem__ Co-authored-by: black-sliver <59490463+black-sliver@users.noreply.github.com> --------- Co-authored-by: black-sliver <59490463+black-sliver@users.noreply.github.com>
What is this fixing or adding?
Title. It seemed weird to me that these aren't really supported by the "OptionDict". I think if you make something that isn't actually a dict, you probably know to override or not use these.
How was this tested?
Put it in The Messenger and generated a few seeds/debugged.
If this makes graphical changes, please attach screenshots.