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

Run labels check only after PR approval #2123

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

andyw8
Copy link
Contributor

@andyw8 andyw8 commented Jun 3, 2024

@andyw8 andyw8 requested a review from a team as a code owner June 3, 2024 17:57
@andyw8 andyw8 requested review from st0012 and vinistock June 3, 2024 17:57
@vinistock vinistock added the chore Chore task label Jun 3, 2024
@andyw8 andyw8 added the server This pull request should be included in the server gem's release notes label Jun 3, 2024
@andyw8 andyw8 enabled auto-merge (squash) June 3, 2024 19:08
@andyw8 andyw8 merged commit 78fbf07 into main Jun 4, 2024
33 of 34 checks passed
@andyw8 andyw8 deleted the andyw8/run-labels-check-on-pr-approval branch June 4, 2024 17:27
@andyw8
Copy link
Contributor Author

andyw8 commented Jun 5, 2024

This might not be working correctly, #2115 seemed get stuck and not run that check, even after approval.

@vinistock
Copy link
Member

Based on the documentation you linked https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#running-a-workflow-when-a-pull-request-is-approved, it seems that the trigger should be on approve and not submit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Chore task server This pull request should be included in the server gem's release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants