Uh oh!
There was an error while loading. Please reload this page.
gh-108303: Remove the non-test Lib/test/reperf.py - #114356
Conversation
gpshead
left a comment
There was a problem hiding this comment.
apparently test_embed.py runs this code?
A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated. Once you have made the requested changes, please leave a comment on this pull request containing the phrase |
sobolevn
commented
Jan 21, 2024
Ok, looks like it was used recently in #110769 by @ambv I propose to:
|
4752216 to
93da596Comparesobolevn
commented
Feb 15, 2024
I am now using another file instead of So, we can remove the unneeded |
encukou
left a comment
There was a problem hiding this comment.
LGTM, but I'd rather not backport the removal.
@encukou looks like this PR #113601 influenced the CI. I fixed tests, so now both Please, take another look :) |
Uh oh!
There was an error while loading. Please reload this page.
encukou
commented
Sep 13, 2024
OK. Thanks! |
Looks like this file was not touched for at least 12 years: https://github.com/python/cpython/commits/main/Lib/test/reperf.py
It does not seem useful. If it is, I can instead move it to
Tool/scripts.Refs #114354