Skip to content

Conversation

@austinvalle
Copy link
Member

Related Issue

N/A

Description

The push event does not run for public forks, so we should switch to using the pull_request target so that we get the "requires approval" dialogue: https://docs.github.com/en/actions/reference/workflows-and-actions/events-that-trigger-workflows#pull-request-events-for-forked-repositories

Rollback Plan

  • If a change needs to be reverted, we will roll out an update to the code within 7 days.

Changes to Security Controls

No

@austinvalle austinvalle requested a review from a team as a code owner September 24, 2025 19:30
Copy link
Contributor

@rainkwan rainkwan left a comment

Choose a reason for hiding this comment

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

Exciting

@austinvalle austinvalle merged commit 157a6a9 into main Sep 24, 2025
5 checks passed
@austinvalle austinvalle deleted the austinvalle-patch-1 branch September 24, 2025 19:33
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants