[Backport release-v1.3] Exclude headref from selected branches #345
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
This is helpful to users to better understand which target branches have been determined to backport to. (cherry picked from commit a01196f)
If we want to exclude the headref from the possible target branches, we need to provide it as a parameter to the findTargetBranches function. I'm not so happy that this affects all previously written unit tests for this function, but it is acceptable because headref has now become important for this function. (cherry picked from commit e6e5284)
This case is relevant where the target_branches input is used to select
the target branches, and the original pull request's headref is part of
it.
For example, consider selecting all feature branches `feature/*` as
target_branches. Glob patterns are not supported yet, but users can find
these with an additional step in their workflow:
```
run: |
branches=$(git branch --list --all | grep 'origin/feature/' | cut -c 18- )
space_delimited=${branches//$'\n'/ }
echo "BRANCHES=${space_delimited}" >> $GITHUB_OUTPUT
```
Note that such a step requires a deep checkout `depth=0`.
But when users do this, they might encounter the feature branch they
just merged, the merged branch that triggered the backport-action.
By excluding it now, we can better support this case. In addition, it
will help support glob patterns in the target_branches input in the
future.
(cherry picked from commit 151736d)
The headref is excluded automatically. Since this is an explicit difference in behavior, it should be clearly documented to avoid surprises for users. As far as I know, there is no use case to backport a pull request to its headref. The cherry-pick would be interrupted because there are no changes which would result in empty commits. Typically, the git user get's to decide to skip these commits, or to abort the cherry-pick. This means that would not be any new commits, and we cannot open an empty pull request. So all of this doesn't work. The only thing left to do, is exclude the headref from the determined target branches. (cherry picked from commit 2f0f6c9)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Backport of #344 to
release-v1.3.