Skip to content

Commit 5f70dd5

Browse files
Merge pull request #29 from weierophinney/hotfix/pull_request_branch
Ensure pull request branch name does not cause collisions
2 parents 4f21db5 + ac9c9d4 commit 5f70dd5

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

entrypoint.sh

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,15 @@ set -e
55
function checkout {
66
local REF=
77
local LOCAL_BRANCH=
8+
local LOCAL_BRANCH_NAME=
89
local BASE_BRANCH=
910

1011
if [[ ! $GITHUB_EVENT_NAME || ! $GITHUB_REPOSITORY || ! $GITHUB_REF ]];then
1112
return
1213
fi
1314

15+
LOCAL_BRANCH_NAME=$GITHUB_HEAD_REF
16+
1417
case $GITHUB_EVENT_NAME in
1518
pull_request)
1619
REF=$GITHUB_REF
@@ -21,6 +24,8 @@ function checkout {
2124
echo "Missing head or base ref env variables; aborting"
2225
exit 1
2326
fi
27+
28+
LOCAL_BRANCH_NAME=pull/${LOCAL_BRANCH_NAME}
2429
;;
2530
push)
2631
REF=${GITHUB_REF/refs\/heads\//}
@@ -54,9 +59,9 @@ function checkout {
5459
echo "Checking out branch ${BASE_BRANCH}"
5560
git checkout ${BASE_BRANCH}
5661
echo "Fetching target ref ${REF}"
57-
git fetch origin ${REF}:${GITHUB_HEAD_REF}
58-
echo "Checking out target ref to ${GITHUB_HEAD_REF}"
59-
git checkout ${GITHUB_HEAD_REF}
62+
git fetch origin ${REF}:${LOCAL_BRANCH_NAME}
63+
echo "Checking out target ref to ${LOCAL_BRANCH_NAME}"
64+
git checkout ${LOCAL_BRANCH_NAME}
6065
fi
6166
}
6267

0 commit comments

Comments
 (0)