Skip to content

Skip self-hosted CI jobs on forks - #1256

Closed
kkebo wants to merge 1 commit into
swiftlang:mainfrom
kkebo:skip-self-hosted-ci-on-fork
Closed

kkebo wants to merge 1 commit into
swiftlang:mainfrom
kkebo:skip-self-hosted-ci-on-fork

Conversation

@kkebo

@kkebo kkebo commented Aug 2, 2026 •

Copy link
Copy Markdown
Member

PR #1241 has added the cmake-smoke-test job, and it requires a self-hosted runner for macOS. However, self-hosted runners are not available for forked repositories. This causes CI to hang and fail after a long period of time on forked repositories. This is frustrating for contributors.

To solve that, I've changed enable_macos_checks of cmake-smoke-test from true to ${{ github.repository == 'swiftlang/swift-format' }}. If the workflow is running in the upstream repository, macOS checks remain enabled. In forked repositories, they are skipped.

Similar PR: swiftlang/swiftly#471

@kkebo
kkebo requested review from hborla and shahmishal as code owners August 2, 2026 14:05
@kkebo
kkebo force-pushed the skip-self-hosted-ci-on-fork branch from 617496d to 9ea69eb Compare August 12, 2026 14:40
@kkebo
kkebo force-pushed the skip-self-hosted-ci-on-fork branch from 9ea69eb to 2ef7322 Compare August 25, 2026 10:29
@kkebo

kkebo commented Aug 25, 2026

Copy link
Copy Markdown
Member Author

Rebased.

@kkebo
kkebo force-pushed the skip-self-hosted-ci-on-fork branch from 2ef7322 to 70c3a21 Compare September 14, 2026 15:34
@kkebo
kkebo force-pushed the skip-self-hosted-ci-on-fork branch from 70c3a21 to af1cdad Compare September 23, 2026 17:48
@kkebo

kkebo commented Sep 23, 2026

Copy link
Copy Markdown
Member Author

Rebased.

@kkebo

kkebo commented Sep 23, 2026

Copy link
Copy Markdown
Member Author

Could you please rerun the failing "Check compatibility with real world projects" job? It looks flaky.

@FranzBusch

Copy link
Copy Markdown
Member

I suggest closing this issue. See: swiftlang/github-workflows#305 (comment)

@kkebo

kkebo commented Oct 8, 2026 •

Copy link
Copy Markdown
Member Author

@FranzBusch Thank you for the information, but I can't remove my fork since it already has its own issues, pull requests, releases, release assets, and etc.

Instead, I'm going to disable some workflows one by one.

https://docs.github.com/en/actions/how-tos/manage-workflow-runs/disable-and-enable-workflows?tool=webui

@kkebo kkebo closed this Oct 8, 2026
@kkebo
kkebo deleted the skip-self-hosted-ci-on-fork branch October 8, 2026 14: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