Skip to content
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.

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

Change preview deployment trigger to pull_request_target #8470

Merged
merged 1 commit into from
Apr 11, 2023

Conversation

dcousens
Copy link
Member

See https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request_target

This should hopefully allow us to (with approval) run the preview deployments for Vercel on (potentially) untrusted pull requests, using the base branch workflows instead of the pull request workflows (which may be modified).

@changeset-bot

This comment was marked as resolved.

@dcousens dcousens requested a review from borisno2 April 11, 2023 05:17
@dcousens dcousens self-assigned this Apr 11, 2023
@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit c6a721d:

Sandbox Source
@keystone-6/sandbox Configuration

@dcousens dcousens merged commit 0503b49 into main Apr 11, 2023
@dcousens dcousens deleted the docs-workflows branch April 11, 2023 05:28
@dcousens
Copy link
Member Author

Actually, this still needed "Required reviewers" on the environment to prevent secret leaks, I think we'll revert

dcousens added a commit that referenced this pull request Apr 11, 2023
dcousens added a commit that referenced this pull request Apr 11, 2023
)

* Revert "change preview trigger to pull_request_target (#8470)"

This reverts commit 0503b49.

* revert to workflow_dispatch only
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