Skip to content

Conversation

@ambv
Copy link
Contributor

@ambv ambv commented Jun 13, 2023

Coverage adds a few frames and the previous limit was causing irrecoverable exceptions before pathlib got to its own support for recursion limit recovery.

@miss-islington
Copy link
Contributor

Thanks @ambv for the PR, and @barneygale for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-105749 is a backport of this pull request to the 3.12 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 13, 2023
…er Coverage (pythonGH-105744)

(cherry picked from commit 4e80082)

Co-authored-by: Łukasz Langa <lukasz@langa.pl>
@bedevere-bot bedevere-bot removed the needs backport to 3.12 only security fixes label Jun 13, 2023
barneygale pushed a commit that referenced this pull request Jun 13, 2023
…der Coverage (GH-105744) (#105749)

gh-102613: Bump recursion limit to fix running test_pathlib under Coverage (GH-105744)
(cherry picked from commit 4e80082)

Co-authored-by: Łukasz Langa <lukasz@langa.pl>
@ambv ambv deleted the pathlib-recursion-limit branch June 13, 2023 21:23
@ambv ambv restored the pathlib-recursion-limit branch June 20, 2023 21:20
@ambv ambv deleted the pathlib-recursion-limit branch November 1, 2023 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir topic-pathlib

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants