Run slow tests on valgrind, but without timeout by radarhere · Pull Request #8975 · python-pillow/Pillow
Navigation Menu
{{ message }}
python-pillow / Pillow Public
- Notifications You must be signed in to change notification settings
- Fork 2.4k
Merged
radarhere merged 2 commits intopython-pillow:mainfrom
May 27, 2025Merged
Run slow tests on valgrind, but without timeout#8975
radarhere merged 2 commits intopython-pillow:mainfrom
Run slow tests on valgrind, but without timeout#8975
radarhere merged 2 commits intopython-pillow:mainfrom
Conversation
Copy link Copy Markdown
Member
radarhere
commented
May 24, 2025
radarhere
commented
It would seem good to me to still let valgrind run its checks if we can. So rather than skipping them completely, run them, but don't timeout.
radarhere
added
the
Testing
label
Copy link Copy Markdown
Member
wiredfool
commented
May 25, 2025
wiredfool commented
May 25, 2025If this works, we should just do this everywhere we're using pytest.mark.timeout.
radarhere
force-pushed
the
valgrind
branch
from
beb4284 to
f054046
Compare
Copy link Copy Markdown
Member Author
radarhere
commented
May 25, 2025
radarhere commented
May 25, 2025Ok, I've pushed a commit for that.
hugovk approved these changes May 26, 2025
radarhere
deleted the
valgrind
branch
radarhere
mentioned this pull request
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