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

Lint and documentation fixes #1

Closed
wants to merge 4 commits into from
Closed

Conversation

radarhere
Copy link

@radarhere radarhere commented Sep 23, 2022

You will see that the Lint CI job is failing at python-pillow#6606.

You will also see that there is an error when building the documentation.

/opt/hostedtoolcache/Python/3.10.6/x64/lib/python3.10/site-packages/PIL/CurImagePlugin.py:docstring of PIL.CurImagePlugin.CurFile:1: WARNING: py:class reference target not found: _io.BytesIO

These are suggestions to resolve that.

@radarhere radarhere changed the title Lint fixes Lint and documentation fixes Sep 23, 2022
jlwoolf pushed a commit that referenced this pull request Sep 27, 2022
Improved palette handling in ImageOps
@jlwoolf jlwoolf closed this Sep 27, 2022
@jlwoolf jlwoolf reopened this Sep 27, 2022
@jlwoolf
Copy link
Owner

jlwoolf commented Sep 27, 2022

Sorry I closed and reopened this. Not the best at using github. Believe these problems should be fixed. I did mean to close it.

@jlwoolf jlwoolf closed this Sep 27, 2022
@radarhere radarhere deleted the jlwoolfmain branch September 27, 2022 22:42
@radarhere
Copy link
Author

You will also see that there is an error when building the documentation.

/opt/hostedtoolcache/Python/3.10.6/x64/lib/python3.10/site-packages/PIL/CurImagePlugin.py:docstring of PIL.CurImagePlugin.CurFile:1: WARNING: py:class reference target not found: _io.BytesIO

This is still a problem, which could fixed by d8f374f

radarhere pushed a commit that referenced this pull request Oct 24, 2022
radarhere pushed a commit that referenced this pull request Dec 30, 2022
radarhere pushed a commit that referenced this pull request Dec 30, 2022
radarhere pushed a commit that referenced this pull request Feb 25, 2023
Use break instead of goto
radarhere pushed a commit that referenced this pull request Feb 25, 2023
radarhere pushed a commit that referenced this pull request Sep 2, 2023
radarhere pushed a commit that referenced this pull request Sep 2, 2023
Call init() if mimetype is not found with preinit()
radarhere pushed a commit that referenced this pull request Sep 2, 2023
Only assert image is similar
radarhere pushed a commit that referenced this pull request Sep 2, 2023
radarhere pushed a commit that referenced this pull request Sep 2, 2023
_decoder.get_next() may return None
radarhere pushed a commit that referenced this pull request Dec 9, 2023
Last trailer comment has priority
radarhere pushed a commit that referenced this pull request Dec 9, 2023
Do not assign new fp attribute to image when closing
radarhere pushed a commit that referenced this pull request Dec 9, 2023
radarhere pushed a commit that referenced this pull request Dec 24, 2023
radarhere pushed a commit that referenced this pull request Feb 24, 2024
radarhere pushed a commit that referenced this pull request Mar 1, 2024
Test error is raised without LOAD_TRUNCATED_IMAGES
radarhere pushed a commit that referenced this pull request Apr 21, 2024
radarhere pushed a commit that referenced this pull request Jul 19, 2024
radarhere pushed a commit that referenced this pull request Jul 19, 2024
radarhere pushed a commit that referenced this pull request Jul 19, 2024
Do not import numpy.typing unless TYPE_CHECKING
radarhere added a commit that referenced this pull request Jul 19, 2024
Co-authored-by: Andrew Murray <[email protected]>
radarhere pushed a commit that referenced this pull request Oct 4, 2024
Renamed variable for first part of splitext to root
radarhere pushed a commit that referenced this pull request Mar 3, 2025
Disable platform guessing instead of adding dependencies-prefix
radarhere pushed a commit that referenced this pull request Mar 3, 2025
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