Skip to content

Conversation

@ManuelHu
Copy link
Contributor

@ManuelHu ManuelHu commented Jul 9, 2025

No description provided.

@gipert
Copy link
Member

gipert commented Jul 9, 2025

ah I just pushed something about this here... #435

btw I don't think your approach will work, you can't do command expansion in CI afaik?

@ManuelHu
Copy link
Contributor Author

ManuelHu commented Jul 9, 2025

btw I don't think your approach will work, you can't do command expansion in CI afaik?

should work, it's just sh/dash not bash, I guess (so most complex expansions do not work)? At least my echo $(nproc) just did its job

@gipert
Copy link
Member

gipert commented Jul 9, 2025

ok, then i am in favor of your solution. can you also include my modifications to the various CMakeLists.txt?

@ManuelHu
Copy link
Contributor Author

ManuelHu commented Jul 9, 2025

sure

@gipert gipert linked an issue Jul 9, 2025 that may be closed by this pull request
@gipert gipert added validation Validation and tests CI/CD labels Jul 9, 2025
@ManuelHu
Copy link
Contributor Author

ManuelHu commented Jul 9, 2025

okay, this removes ~10 minutes from CI run time, great!

@ManuelHu ManuelHu merged commit ccd3a5e into main Jul 9, 2025
8 checks passed
@ManuelHu ManuelHu deleted the ci-nproc branch July 9, 2025 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD validation Validation and tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Optimize test suite resource usage on GitHub CI runners

3 participants