Skip to content

Remove self-hosted runners #3724

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Nov 13, 2025
Merged

Remove self-hosted runners #3724

merged 2 commits into from
Nov 13, 2025

Conversation

@vedhavyas
Copy link
Contributor

@vedhavyas vedhavyas commented Nov 7, 2025 •
edited
Loading

Removes self-hosted runners

  • Disabled windows cache as this is not helpful. Maybe we can revisit this in the future.
  • Disabled macos deps cache for macos in clippy. For some reason, cached deps seems to break hwlocal. Need to figure out if this is due to change in envs from selhosted to github runners

@jfrank-summit

Code contributor checklist:

@vedhavyas vedhavyas changed the title remove self-hosted runners(wip) Remove self-hosted runners Nov 13, 2025
@vedhavyas vedhavyas marked this pull request as ready for review November 13, 2025 10:03
@vedhavyas vedhavyas requested a review from nazar-pc as a code owner November 13, 2025 10:03
@immunefi-magnus
Copy link

immunefi-magnus bot commented Nov 13, 2025

🛡️ Immunefi PR Reviews

We noticed that your project isn't set up for automatic code reviews. If you'd like this PR reviewed by the Immunefi team, you can request it manually using the link below:

🔗 Send this PR in for review

Once submitted, we'll take care of assigning a reviewer and follow up here.

@vedhavyas vedhavyas enabled auto-merge November 13, 2025 11:14
jfrank-summit
jfrank-summit approved these changes Nov 13, 2025
View reviewed changes
Copy link
Member

@jfrank-summit jfrank-summit left a comment

Choose a reason for hiding this comment

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

Are the runs-on actions still needed/used?

@vedhavyas vedhavyas added this pull request to the merge queue Nov 13, 2025
@vedhavyas
Copy link
Contributor Author

vedhavyas commented Nov 13, 2025

Are the runs-on actions still needed/used?

Not used in this repository anymore once this PR is in

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 13, 2025
@vedhavyas vedhavyas added this pull request to the merge queue Nov 13, 2025
Merged via the queue into main with commit 3cd3697 Nov 13, 2025
15 checks passed
@vedhavyas vedhavyas deleted the gh_actions branch November 13, 2025 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jfrank-summit jfrank-summit jfrank-summit approved these changes

@nazar-pc nazar-pc Awaiting requested review from nazar-pc nazar-pc is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vedhavyas @jfrank-summit