Skip to content

MNT: Execute slowest tests on schedule #258

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jhlegarreta
Copy link

@jhlegarreta jhlegarreta commented Jul 21, 2025

Execute slowest tests on schedule.

Fixes #229.

Execute slowest tests on schedule.
@jhlegarreta
Copy link
Author

Copy link
Collaborator

@oesteban oesteban left a comment

Choose a reason for hiding this comment

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

I dropped 3.9 in #259 because I believe version 25.x series should drop it (and backward compatibility of #256 was really hard).

That all said, what takes the most time here is CircleCI's tests. Perhaps we can drop all the docker shebang which substantially adds to the build.

Drop 3.9 from tested Python version matrix.

Co-authored-by: Oscar Esteban <[email protected]>
@jhlegarreta jhlegarreta force-pushed the mnt/mark-tests-slow branch from 43b21f7 to 8a2071a Compare July 22, 2025 23:34
@jhlegarreta
Copy link
Author

That all said, what takes the most time here is CircleCI's tests. Perhaps we can drop all the docker shebang which substantially adds to the build.

Please, feel free to take over; I thought that issue #229 referred to Python testing purely, so did not have a look at what are other bottlenecks. I am not sure the current approach is working well either, since I do not see the GitHub checks taking place on the commit event.

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.

Mark tests as slow to enable separate running
2 participants