Skip to content

MAINT: Use pytest-xdist in CI - #91

Merged
derb12 merged 1 commit into
developmentfrom
ci_xdist
Aug 7, 2026
Merged

derb12 merged 1 commit into
developmentfrom
ci_xdist

Conversation

@derb12

@derb12 derb12 commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Description

Just a test to see if running pytest-xdist with 4 processes runs fine or if it's too much for github actions. Running locally with act shows a time improvement even with the limited resources of one machine.

Type of Pull Request

  • Bug Fix
  • New Feature
  • Miscellaneous Changes (refactor, code improvements, etc.)
  • Documentation or Example Programs

Pull Request Checklist

  • New code and/or documentation is valid for use with the BSD 3-clause license.
  • New code is fully documented with docstrings that follow Numpy style,
    if applicable.
  • New code follows PEP 8 standards as closely as possible, if applicable.
  • Added/updated tests and ensured they pass locally, if applicable.
  • Verified that documentation builds locally, if applicable.

Use 4 processes since Github runners have 4 cores, I think.
@derb12

derb12 commented Aug 7, 2026

Copy link
Copy Markdown
Owner Author

Comparing to the last PR where actions ran against development in #84, affected workflows decreased by ~2-3 minutes for non free-threaded and ~4 minutes for py314t, so seems like it works fine.

@derb12
derb12 merged commit 3d43e11 into development Aug 7, 2026
11 checks passed
@derb12
derb12 deleted the ci_xdist branch August 7, 2026 22:16
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