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.

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

Auto detect CPU cores in CI #125

Merged
merged 4 commits into from
Feb 1, 2023
Merged

Auto detect CPU cores in CI #125

merged 4 commits into from
Feb 1, 2023

Conversation

lilleyse
Copy link
Contributor

Follow up from #124 (comment)

  • Detects the number of logical CPU cores automatically using github-actions-cpu-cores. In a separate commit I echoed the count and got the expected values: 2 for linux and 8 for windows (only windows is using our AWS infrastructure).
  • Upgraded all the other third party github actions

@kring
Copy link
Member

kring commented Jan 31, 2023

Apparently even d:\cesium-omniverse\cesium-omniverse\.conan is too long for Conan builds to succeed. The only reason it seemed ok before was because the cache was populated successfully with a shorter path, and so it didn't need to build with the longer one. So I've commited a change to move it back to d:\.conan instead.

Copy link
Contributor

@weegeekps weegeekps left a comment

Choose a reason for hiding this comment

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

:shipit:

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.

3 participants