Skip to content

Gate PR preview deploy behind org membership check#38

Merged
nicolethoen merged 1 commit into
mainfrom
update-pr-preview
Apr 8, 2026
Merged

Gate PR preview deploy behind org membership check#38
nicolethoen merged 1 commit into
mainfrom
update-pr-preview

Conversation

@nicolethoen

Copy link
Copy Markdown

Summary

  • Adds a check-permissions job that verifies the PR author (or commenter) is an OWNER, MEMBER, or COLLABORATOR before running any steps that use secrets
  • Adds /deploy-preview comment trigger so org members can request a preview on any PR
  • Switches to the shared patternfly/.github/.github/actions/surge-preview@main action
  • Updates actions/checkout from v2 to v4 and adds yarn dependency caching
  • Removes lint steps from preview workflow (already covered by check-pr.yml)

Aligns with other repos in the extensions-testing org (e.g. react-console).

Test plan

  • Verify org member PRs still get automatic preview deploys
  • Verify external contributor PRs skip the deploy-preview job
  • Verify /deploy-preview comment from an org member triggers a preview
  • Verify basic CI (check-pr.yml) still runs for all PRs regardless of author

🤖 Generated with Claude Code

Only deploy previews for PRs from org members/collaborators, preventing
untrusted PRs from accessing secrets. Adds /deploy-preview comment
trigger for on-demand previews. Aligns with other extension-testing repos.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@patternfly-build

Copy link
Copy Markdown

@nicolethoen nicolethoen merged commit f19aadd into main Apr 8, 2026
5 checks passed
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