Skip to content

Relax all getrefcount tests to allow lower numbers#854

Merged
ofek merged 1 commit intojcrist:mainfrom
hroncok:getrefcount
Oct 20, 2025
Merged

Relax all getrefcount tests to allow lower numbers#854
ofek merged 1 commit intojcrist:mainfrom
hroncok:getrefcount

Conversation

@hroncok
Copy link
Copy Markdown
Contributor

@hroncok hroncok commented May 26, 2025

This is related to https://docs.python.org/3.14/whatsnew/3.14.html#whatsnew314-refcount

Created via:

sed -Ei 's/sys\.getrefcount\(([^\)]+)\) == ([0-9]+)/sys.getrefcount(\1) <= \2/' tests/test_*.py

Perhaps this is too strong a hammer, but it makes the tests pass on 3.14.

This is related to https://docs.python.org/3.14/whatsnew/3.14.html#whatsnew314-refcount

Created via:

    sed -Ei 's/sys\.getrefcount\(([^\)]+)\) == ([0-9]+)/sys.getrefcount(\1) <= \2/' tests/test_*.py

Perhaps this is too strong a hammer, but it makes the tests pass on 3.14.
@ofek ofek marked this pull request as ready for review October 20, 2025 04:26
Copy link
Copy Markdown
Collaborator

@ofek ofek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't think of a much better way than this, thanks!

@ofek ofek merged commit a365c4b into jcrist:main Oct 20, 2025
manzt pushed a commit to marimo-team/msgspec that referenced this pull request Nov 14, 2025
manzt added a commit to marimo-team/msgspec that referenced this pull request Nov 14, 2025
* Fix crash when TypedDict contains incorrect metadata (jcrist#853)

* Fix annotations support on 3.14 (jcrist#852)

* Relax all getrefcount tests to allow lower numbers (jcrist#854)

* Add 3.14 to CI (jcrist#888)

* add free-threading support  (jcrist#877)

Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>

* Enable Python 3.14 tests in CI

Remove cp314-* from CIBW_TEST_SKIP to allow Python 3.14 tests to run.
Previously, 3.14 wheels were built but tests were skipped.

---------

Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
Co-authored-by: Miro Hrončok <miro@hroncok.cz>
Co-authored-by: Ofek Lev <ofekmeister@gmail.com>
Co-authored-by: Kumar Aditya <kumaraditya@python.org>
Sign up for free to 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.

2 participants