Skip to content
This repository was archived by the owner on Dec 20, 2023. It is now read-only.

Typing #68

Merged
merged 8 commits into from
Nov 7, 2023
Merged

Typing #68

merged 8 commits into from
Nov 7, 2023

Conversation

tiptenbrink
Copy link
Collaborator

@tiptenbrink tiptenbrink commented Nov 7, 2023

PyCharm's Python language server doesn't really handle the decorators, so instead I tried VS Code, which uses Pyright for the Pylance plugin, which is much better. To actually exhaustively typecheck, I finally added types everywhere to support mypy. This uncovered some issues, so this also includes some refactoring.

Most notably, the way that the userdata subclassing from how it was used in auth (which needs it for the ID token) has been changed. There is now the IdUserData class, which contains all necessary behavior. This prevented a lot of generic magic that would have been necessary to support the old way in a type-safe manner.

Tests still need to be fixed.

Closes #41.

@tiptenbrink tiptenbrink merged commit c88160a into main Nov 7, 2023
@tiptenbrink tiptenbrink deleted the typing branch November 29, 2023 08:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mypy across entire project
1 participant