-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
add label back when conflicts are resolved if the label was removed #88
Comments
You're saying it doesn't add |
That is correct
File locations:
The dummy workflow ensures that the workflow also works on pr opened on forks. If u dont include this workflow PRs from forks get a github API error. |
Ah now I remember. I think this was intended since the label that's removed could be something like "ready to merge". And rebase after review kind of invalidates that. But that depends on the semantics of the labels so an option to add the Thanks for adding the PRs. Made it clearer why you'd want that behavior. Feel free to send a PR adding such an option. |
Unfortunately i have no experience with GH actions. So cant help u with that :( |
Just a thought. For our usecase/workflow it make totally sense but maybe for others not. Only add the label back when it was removed. |
Is it actually possible for the action to add back a label "only if it was removed"? Where would that state be kept? I was initially thinking that, to get behavior that's flexible enough to handle everyone's use cases, there would need to be But even that doesn't cover the case where the action should remove and restore externally-created labels, but only the ones it's previously placed/removed. That would require the action keeping a history of its own changes somewhere, which (correct me if I'm wrong) I don't think it currently does? |
When
removeOnDirtyLabel
removes the label it doesnt add the label back when the conflicts are resolved. It would be nice for the workkflow to do that also.If the label wasn't present when the conflicts occurred the workflow doesn't have to apply a label.
The text was updated successfully, but these errors were encountered: