Skip to content

Comments

Replace Any with Incomplete in distutils#12842

Merged
srittau merged 4 commits intopython:mainfrom
Avasam:use-Incomplete-in-distutils
Oct 18, 2024
Merged

Replace Any with Incomplete in distutils#12842
srittau merged 4 commits intopython:mainfrom
Avasam:use-Incomplete-in-distutils

Conversation

@Avasam
Copy link
Collaborator

@Avasam Avasam commented Oct 17, 2024

Works towards #9550

I'm working on providing distutils stubs directly from setuptools (for Python 3.12+ support). And I wanted to start from as clean a slate as possible when copying typeshed's stubs. It includes not using Any where not necessary.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Co-authored-by: Sebastian Rittau <[email protected]>
@Avasam Avasam requested a review from srittau October 18, 2024 15:14
@github-actions
Copy link
Contributor

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

@srittau srittau merged commit de4842d into python:main Oct 18, 2024
@Avasam Avasam deleted the use-Incomplete-in-distutils branch October 18, 2024 16:29
fullname: str = ...,
**attrs,
# Custom Distributions could accept more params
**attrs: Any,
Copy link
Contributor

Choose a reason for hiding this comment

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

Just came across this commit while investigating why pyright started warning that Type of "setup" is partially unknown since release of 1.1.375 a few weeks ago. Hopefully, this slight addition should fix it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sounds like it would, yeah.

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