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 type hints to Image.py #7924

Merged
merged 3 commits into from
Apr 1, 2024
Merged

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Mar 31, 2024

Still some to do later.

src/PIL/Image.py Outdated Show resolved Hide resolved
src/PIL/Image.py Outdated Show resolved Hide resolved
@hugovk hugovk force-pushed the image-type-hints branch from 397de28 to 8c9621b Compare March 31, 2024 19:04
Co-authored-by: Ondrej Baranovič <ondreko.tiba@gmail.com>
@hugovk hugovk force-pushed the image-type-hints branch from 8c9621b to b6c755d Compare March 31, 2024 19:05
src/PIL/Image.py Outdated Show resolved Hide resolved
src/PIL/Image.py Outdated Show resolved Hide resolved
src/PIL/Image.py Outdated Show resolved Hide resolved
src/PIL/Image.py Outdated Show resolved Hide resolved
Co-authored-by: Andrew Murray <3112309+radarhere@users.noreply.github.com>
src/PIL/Image.py Outdated Show resolved Hide resolved
src/PIL/Image.py Outdated Show resolved Hide resolved
Co-authored-by: Andrew Murray <3112309+radarhere@users.noreply.github.com>
@hugovk hugovk added the automerge Automatically merge PRs that are ready label Apr 1, 2024
@Yay295
Copy link
Contributor

Yay295 commented Apr 1, 2024

I think ImagingCore would be typed as typing.Any, pending a more concrete definition. Though I suppose leaving it untyped for now works as well.

@mergify mergify bot merged commit 835ca1b into python-pillow:main Apr 1, 2024
56 of 57 checks passed
@hugovk hugovk deleted the image-type-hints branch April 1, 2024 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Automatically merge PRs that are ready Type hints
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants