Skip to content

Switch Basilisp test suite to use only 2 CPUs#876

Merged
jeaye merged 3 commits into
jank-lang:mainfrom
basilisp-lang:fix/basilisp-test-run
May 9, 2026
Merged

Switch Basilisp test suite to use only 2 CPUs#876
jeaye merged 3 commits into
jank-lang:mainfrom
basilisp-lang:fix/basilisp-test-run

Conversation

@chrisrink10
Copy link
Copy Markdown
Contributor

@chrisrink10 chrisrink10 commented May 9, 2026

pytest-xdist's auto option can often be slower than setting a specific number of CPUs. I think standard Github Actions workers only have 2 vCPUs so this should end up actually being faster than using auto.

I also bumped the different Github Actions versions to use newer releases which use Node 24 to suppress some warnings during the runs.

@chrisrink10 chrisrink10 marked this pull request as ready for review May 9, 2026 19:01
Comment thread .github/workflows/ci.yml
- uses: actions/checkout@v6

- name: Prepare dotnet
uses: xt0rted/setup-dotnet@v1.5.0
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure why this action was selected, but it appears to be out of sync from the official setup-dotnet action from Github and hasn't been updated since 2022.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@dmiller Any reason to not use the official setup-dotnet?

@jeaye jeaye merged commit 1cbab0f into jank-lang:main May 9, 2026
6 checks passed
@jeaye
Copy link
Copy Markdown
Member

jeaye commented May 9, 2026

Thanks for the speedup, Chris!

@chrisrink10 chrisrink10 deleted the fix/basilisp-test-run branch May 10, 2026 02:30
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.

2 participants