Skip to content
This repository has been archived by the owner on Nov 28, 2023. It is now read-only.

Remove label-check from merge_group #125

Merged
merged 1 commit into from
Oct 3, 2023
Merged

Remove label-check from merge_group #125

merged 1 commit into from
Oct 3, 2023

Conversation

romancardenas
Copy link
Contributor

It seems that the action used to check forbidden labels only works for PRs. Removing it from merge_group.

It seems that the action used to check forbidden labels only works for PRs. Removing it from merge_group.
@romancardenas romancardenas requested a review from a team as a code owner October 3, 2023 13:47
@romancardenas romancardenas added this pull request to the merge queue Oct 3, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Oct 3, 2023
@MabezDev MabezDev added this pull request to the merge queue Oct 3, 2023
@romancardenas
Copy link
Contributor Author

I forgot to remove the label-check requirement in settings and the merge queue hanged. Let's see if it is solved now.

BTW, I'll explore alternative CI workflows for this feature, the current limitations make this action a bit useless/annoying

Merged via the queue into master with commit 28b916d Oct 3, 2023
@romancardenas romancardenas deleted the label-patch branch October 4, 2023 07:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants