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.

Sign up for GitHub

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

Jump to bottom

CI: run tests also with GraalPy 24.0 #605

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

CI: run tests also with GraalPy 24.0 #605

wants to merge 2 commits into from

Conversation

dnicolodi
Copy link
Member

No description provided.

@dnicolodi
Copy link
Member Author

dnicolodi commented Mar 24, 2024 •
edited
Loading

The tests do not pass with GraalPy but pytest is weird on this platform making debugging the problem not really pleasant and I don't really know if it is worth it. At a quick look, it seems that we generate wheel tags that are not compatible with the ones generated by packaging. It also looks like that GraalPy, at least in the version installed on the GitHub Action runners, patches packaging. To add to the fun, the test suites takes about 30 times more to run on GraalPy than on CPython. Unless someone asks for it, I'm not really motivated to untangle this.

@rgommers
Copy link
Contributor

Agreed. I know of GraalPy, but haven't encountered it in real-world situations yet.

@rgommers rgommers added the CI Continuous Integration label Mar 24, 2024
@dnicolodi dnicolodi marked this pull request as draft March 24, 2024 19:21
@dnicolodi
CI: run tests also with GraalPy 24.0
67f6882
@dnicolodi
TST: ignore distutils deprecation warning
e1699e9 
Somehow this warning is triggered when running the tests with GraalPy.
@dnicolodi dnicolodi changed the title CI: extend matrix CI: run tests also with GraalPy 24.0 Mar 24, 2024
@dnicolodi
Copy link
Member Author

Yeah. This actually started with me checking that we run the tests on PyPy 3.10 after I saw mesonbuild/meson#12979 and noticing that GitHub Actions offer GraalPy too. For the records, we run the tests with PyPy 3.10.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CI Continuous Integration
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dnicolodi @rgommers