FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Improve `oauthlib.oauth2.rfc6749` by donbarbos · Pull Request #13965 · python/typeshed · GitHub

Repository navigation

Improve oauthlib.oauth2.rfc6749 - #13965

Merged
srittau merged 5 commits into
python:mainfrom
donbarbos:improve-oauthlib-oauth2-rfc6749-3
May 13, 2025
Merged

srittau merged 5 commits into
python:mainfrom
donbarbos:improve-oauthlib-oauth2-rfc6749-3

Conversation

Copy link
Copy Markdown
Contributor

No description provided.

This comment has been minimized.

Comment on lines +34 to +35
*args: Any, # actually, these are not used
**kwargs: Any, # actually, these are not used

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

If they are not used, we should use (_typeshed.)Unused instead of a comment. A potentially better option is to remove the arguments to prevent users from misspelling, e.g. Server(token_genarator=...) isn't an error with the current annotations.

donbarbos May 13, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

to be honest i already forgot why i preferred Any, but i think because subclasses can use them

and still I'll probably just remove them

Copy link
Copy Markdown
Contributor

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

srittau merged commit fd20f4a into python:main May 13, 2025
mmingyu pushed a commit to mmingyu/typeshed that referenced this pull request May 16, 2025
donbarbos deleted the improve-oauthlib-oauth2-rfc6749-3 branch February 25, 2026 04:08
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL