Skip to content

Prevent fork pull requests from running CI - #22

Closed
erseco wants to merge 1 commit into
mainfrom
security/block-fork-ci
Closed

Prevent fork pull requests from running CI#22
erseco wants to merge 1 commit into
mainfrom
security/block-fork-ci

Conversation

@erseco

@erseco erseco commented Jul 24, 2026

Copy link
Copy Markdown
Member

Summary

  • skip the PHP lint and test job for pull requests originating from forks
  • preserve pushes and same-repository pull requests

Security rationale

The workflow installs Composer dependencies and executes repository code. Restricting fork-originated jobs prevents untrusted code from consuming runner and network resources.

@github-actions

Copy link
Copy Markdown

omeka-s Playground Preview

Open this PR in Omeka-S Playground
Try this PR in your browser

This preview loads the plugin directly from the PR branch ZIP in Omeka-S Playground.

erseco commented Jul 24, 2026

Copy link
Copy Markdown
Member Author

Closing this after reviewing the GitHub Actions approval model. Normal pull_request workflows will remain available for legitimate fork contributions and will rely on the repository or organization setting Require approval for all external contributors. This avoids blocking approved external contributions while preserving the manual approval gate.

@erseco erseco closed this Jul 24, 2026
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.

1 participant