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

Add types to verify_token and Request __init__ based on comments in the source code. #1588

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

wyattscarpenter
Copy link

Recently, I've been using this library with mypy strict mode, which doesn't like the fact that these functions are unannotated (they're the only functions I use, and thus the only ones mypy complains about to me). I'm happy to see that this project has py.typed, so these annotations should fix the problem!

@wyattscarpenter wyattscarpenter requested review from a team as code owners August 20, 2024 08:17
@arithmetic1728 arithmetic1728 added the kokoro:run Add this label to force Kokoro to re-run the tests. label Aug 20, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:run Add this label to force Kokoro to re-run the tests. label Aug 20, 2024
@arithmetic1728
Copy link
Contributor

arithmetic1728 commented Aug 20, 2024

Could you squash your commits, and change the commit message tofix: add types to verify_token and request __init__? The conventionalcommits.org test requires the commit message to start with the "fix:" prefix. Thanks!

@wyattscarpenter
Copy link
Author

Happy to!

Recently, I've been using this library with mypy strict mode, which doesn't like the fact that these functions are unannotated (they're the only functions I use, and thus the only ones mypy complains about to me). I'm happy to see that this project has py.typed, so these annotations should fix the problem!
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