Skip to content

tests: avoid immortal objects in tests - #5150

Merged
henryiii merged 1 commit into
pybind:masterfrom
henryiii:henryiii/tests/gilup
May 30, 2024
Merged

tests: avoid immortal objects in tests#5150
henryiii merged 1 commit into
pybind:masterfrom
henryiii:henryiii/tests/gilup

Conversation

@henryiii

@henryiiihenryiii commented May 30, 2024

Copy link
Copy Markdown
Collaborator

Description

Follow up #5139, from #5148. Avoid immortal objects so we can track the refcounts.

Also actually compiles against 3.13t in tests, missed that in #5139, though I was doing it locally.

Suggested changelog entry:

* Avoid immortal objects in tests

Co-authored-by: Sam Gross <colesbury@gmail.com>
Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
@henryiii
henryiiiforce-pushed the henryiii/tests/gilup branch from 0017011 to 08e508fCompareMay 30, 2024 03:32
@henryiii

Copy link
Copy Markdown
CollaboratorAuthor

This is just a test fix for my previous PR, so will merge to rebase #5148. Can revert if there's an issue!

@henryiii
henryiii merged commit 1a0ff40 into pybind:masterMay 30, 2024
@henryiii
henryiii deleted the henryiii/tests/gilup branch May 30, 2024 05:40
@github-actionsgithub-actionsBot added the needs changelog Possibly needs a changelog entry label May 30, 2024
@henryiiihenryiii removed the needs changelog Possibly needs a changelog entry label Jun 26, 2024
Sign up for freeto 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.

1 participant

@henryiii