Skip to content

🔧use github-hosted runners#10

Merged
juiwenchen merged 3 commits into
mainfrom
update-CI-nodes
Aug 1, 2025
Merged

🔧use github-hosted runners#10
juiwenchen merged 3 commits into
mainfrom
update-CI-nodes

Conversation

@juiwenchen
Copy link
Copy Markdown
Contributor

🔧 update runners to GitHub hosted ones

- use github-hosted runners
- not specified cache dir in macos
@juiwenchen juiwenchen requested a review from ubmarco July 31, 2025 09:59
Comment thread .github/actions/setup_rye/action.yml Outdated
if: runner.os == 'macOS'
run: echo "UV_CACHE_DIR=/Users/gh-runner/Library/Caches/uv" >> $GITHUB_ENV
shell: bash
# The cache dir cannot be created in MacOs in GitHub hosted runner because of lack of permission
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.

No comment, just delete

@juiwenchen juiwenchen requested a review from ubmarco August 1, 2025 07:45
Comment thread .github/actions/setup_rye/action.yml Outdated
if: runner.os == 'macOS'
run: echo "UV_CACHE_DIR=/Users/gh-runner/Library/Caches/uv" >> $GITHUB_ENV
shell: bash
- name: Set UV_CACHE_DIR for Windows
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.

The other cache settings are also not required for the temporary public runners. The containers are destroyed after the run anyway.

@juiwenchen juiwenchen requested a review from ubmarco August 1, 2025 08:23
@juiwenchen juiwenchen merged commit 8b33b8d into main Aug 1, 2025
8 checks passed
@juiwenchen juiwenchen deleted the update-CI-nodes branch August 1, 2025 09:40
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