-
-
Notifications
You must be signed in to change notification settings - Fork 34.4k
_testcapi should be multi-phase #139098
Copy link
Copy link
Closed
Labels
extension-modulesC modules in the Modules dirC modules in the Modules dirsprinttestsTests in the Lib/test dirTests in the Lib/test dirtype-refactorCode refactoring (with no changes in behavior)Code refactoring (with no changes in behavior)
Metadata
Metadata
Assignees
Labels
extension-modulesC modules in the Modules dirC modules in the Modules dirsprinttestsTests in the Lib/test dirTests in the Lib/test dirtype-refactorCode refactoring (with no changes in behavior)Code refactoring (with no changes in behavior)
Projects
Status
Done
_testcapiuses legacy single-phase initialization, which prevents us from being able to use it in subinterpreter tests. Historically,_testcapiwas kept as single-phase because we needed a module to test single-phase init, but since we now have_testsinglephase, we can move it to multi-phase init.This is currently blocking #136004, as
_testcapiis needed for one of the tests.Linked PRs
_testcapi#139102