Skip to content

Restrict and notify workflow runs on fork #11

Merged
soumeh01 merged 6 commits into
mainfrom
block-runs
Jul 6, 2026
Merged

Restrict and notify workflow runs on fork #11
soumeh01 merged 6 commits into
mainfrom
block-runs

Conversation

@soumeh01

@soumeh01 soumeh01 commented Jun 30, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator
  • Fail immediately when a forked repository attempts to run a workflow requiring a custom runner, instead of waiting for a timeout with the following message:

⚠️ AppKit-E8 hardware test was not run
Workflows on the private runner are restricted for security reasons.

  • Workflows triggered by users with direct access to the repository (members and collaborators) are allowed to run on the custom runner.

Evagan2018
Evagan2018 previously approved these changes Jun 30, 2026
View reviewed changes

@Evagan2018 Evagan2018 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@soumeh01 soumeh01 marked this pull request as ready for review June 30, 2026 09:11
@soumeh01 soumeh01 requested a review from jkrech June 30, 2026 09:11
@soumeh01 soumeh01 changed the title Block Runs Restrict and notify workflow runs on fork Jun 30, 2026
@soumeh01 soumeh01 requested a review from Evagan2018 July 6, 2026 06:34
Evagan2018
Evagan2018 approved these changes Jul 6, 2026
View reviewed changes

@Evagan2018 Evagan2018 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@soumeh01 soumeh01 merged commit 8c385d4 into main Jul 6, 2026
12 checks passed
@soumeh01 soumeh01 deleted the block-runs branch July 6, 2026 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Evagan2018 Evagan2018 Evagan2018 approved these changes
@JonatanAntoni JonatanAntoni Awaiting requested review from JonatanAntoni
@jkrech jkrech Awaiting requested review from jkrech

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@soumeh01 @Evagan2018