Skip to content

Commit

Permalink
Merge pull request containers#3190 from edsantiago/master
Browse files Browse the repository at this point in the history
pr-should-include-tests: try to make work in buildah
  • Loading branch information
openshift-merge-robot authored Apr 29, 2021
2 parents fcb2f13 + 6587442 commit df864f1
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion tests/validate/pr-should-include-tests
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,16 @@ fi
# HEAD should be good enough, but the CIRRUS envariable allows us to test
head=${CIRRUS_CHANGE_IN_REPO:-HEAD}
# Base of this PR. Here we absolutely rely on cirrus.
base=$(git merge-base ${DEST_BRANCH:-master} $head)
base=$(git merge-base ${GITVALIDATE_EPOCH:-master} $head)

# Sanity check:
if [[ -z "$base" ]]; then
echo "$(basename $0): internal error: could not determine merge-base"
echo " head = $head"
echo " CIRRUS_CHANGE_IN_REPO = $CIRRUS_CHANGE_IN_REPO"
echo " GITVALIDATE_EPOCH = $GITVALIDATE_EPOCH"
exit 1
fi

# This gives us a list of files touched in all commits, e.g.
# A foo.c
Expand Down

0 comments on commit df864f1

Please sign in to comment.