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

refactor(framework:skip) Hide private TypedDict members and inherit from MutableMapping #3218

Merged
merged 18 commits into from
Aug 15, 2024

Conversation

panh99
Copy link
Contributor

@panh99 panh99 commented Apr 4, 2024

After inheriting MutableMapping, our TypedDict can be recognized by IDE, mypy and pylint.

@panh99 panh99 changed the title Hide check_key_fn and check_value_fn of TypedDict. Hide check_key_fn and check_value_fn of TypedDict Jun 21, 2024
@panh99 panh99 changed the title Hide check_key_fn and check_value_fn of TypedDict refactor(framework:skip) Hide check_key_fn and check_value_fn of TypedDict Jun 21, 2024
@panh99 panh99 changed the title refactor(framework:skip) Hide check_key_fn and check_value_fn of TypedDict refactor(framework:skip) Hide check_key_fn and check_value_fn of TypedDict and make the class inherit from MutableMapping Jun 21, 2024
@panh99 panh99 changed the title refactor(framework:skip) Hide check_key_fn and check_value_fn of TypedDict and make the class inherit from MutableMapping refactor(framework:skip) Hide private class variables of TypedDict and make the class inherit from MutableMapping Jun 21, 2024
@panh99 panh99 marked this pull request as ready for review June 21, 2024 19:10
Copy link
Contributor

@jafermarq jafermarq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

@danieljanes danieljanes changed the title refactor(framework:skip) Hide private class variables of TypedDict and make the class inherit from MutableMapping refactor(framework:skip) Hide private TypedDict members and inherit from MutableMapping Aug 15, 2024
@danieljanes danieljanes enabled auto-merge (squash) August 15, 2024 11:59
@danieljanes danieljanes merged commit 4e617a3 into main Aug 15, 2024
34 checks passed
@danieljanes danieljanes deleted the hide-members-records branch August 15, 2024 12:01
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.

3 participants