Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TST: add CPython 3.13 to regular test matrix #525

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

neutrinoceros
Copy link
Member

@neutrinoceros neutrinoceros commented Oct 8, 2024

@neutrinoceros neutrinoceros added this to the 3.0.4 milestone Oct 8, 2024
@neutrinoceros
Copy link
Member Author

neutrinoceros commented Oct 8, 2024

Looks like we're hitting a known issue from pint
xref hgrecco/pint#1969

I don't want to be blocked by development of an optional dependency that we merely want to interface with, so I'll be looking for a way to skip pint tests on cp313 for now.

@neutrinoceros
Copy link
Member Author

neutrinoceros commented Oct 8, 2024

I actually don't know of a good solution. The only one I can think of is skipping all doctests on py313 for now, but then it's not ready to replace py312 entierely on CI and I don't want to add jobs just to get around a bug in pint. So, unless the upstream issue is promptly resolved, I'll just not include this in unyt 3.0.4

@neutrinoceros
Copy link
Member Author

pint 0.24.4 was just released, let's try this again

@neutrinoceros neutrinoceros marked this pull request as ready for review November 7, 2024 16:40
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.

1 participant