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

[MNT] set number of pytest-xdist workers to auto #2992

Merged
merged 1 commit into from
Jul 13, 2022
Merged

[MNT] set number of pytest-xdist workers to auto #2992

merged 1 commit into from
Jul 13, 2022

Conversation

fkiraly
Copy link
Collaborator

@fkiraly fkiraly commented Jul 13, 2022

This PR sets the number of pytest-xdist workers to auto, which will use max.
This is 3 on mac, and 2 on the other OS
https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners
so this should cut down time on mac to 2/3.

Related PR: #2890.

@fkiraly fkiraly added the maintenance Continuous integration, unit testing & package distribution label Jul 13, 2022
@fkiraly fkiraly requested a review from aiwalter as a code owner July 13, 2022 09:07
@lmmentel lmmentel self-requested a review July 13, 2022 12:33
@lmmentel lmmentel merged commit 9276d32 into main Jul 13, 2022
@lmmentel lmmentel deleted the n-auto branch July 13, 2022 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lmmentel lmmentel lmmentel approved these changes

@aiwalter aiwalter Awaiting requested review from aiwalter

Assignees
No one assigned
Labels
maintenance Continuous integration, unit testing & package distribution
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fkiraly @lmmentel