Skip to content

Commit

Permalink
Reduce Ubuntu and MacOS runners from 4 to 3
Browse files Browse the repository at this point in the history
The Windows runners are the pytest bottleneck by a margin, so not much
is gained by having the 4th runners. This way we reduce the usage. This
is probably helpful when runners are scarce.
  • Loading branch information
maresb committed Sep 13, 2024
1 parent cae2e2c commit bdf4217
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,8 @@ jobs:

# Prune the indices so that we only run the splits up to the group size
# defined below in "include". (This is ugly but effective.)
- os: ubuntu
pytest-split-group-index: 4
- os: ubuntu
pytest-split-group-index: 5
- os: ubuntu
Expand All @@ -55,6 +57,8 @@ jobs:
pytest-split-group-index: 9
- os: ubuntu
pytest-split-group-index: 10
- os: macos
pytest-split-group-index: 4
- os: macos
pytest-split-group-index: 5
- os: macos
Expand All @@ -69,9 +73,9 @@ jobs:
pytest-split-group-index: 10
include:
- os: ubuntu
pytest-split-group-size: 4
pytest-split-group-size: 3
- os: macos
pytest-split-group-size: 4
pytest-split-group-size: 3
- os: windows
pytest-split-group-size: 10

Expand Down

0 comments on commit bdf4217

Please sign in to comment.