Skip to content

Commit

Permalink
tests: disable apidiff check (#316)
Browse files Browse the repository at this point in the history
  • Loading branch information
vangent authored Aug 31, 2021
1 parent 7e8af29 commit 3b6fb78
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
4 changes: 2 additions & 2 deletions internal/check_api_change.sh
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,11 @@
# This script expects:
# a) to be run at the root of the repository
# b) HEAD is pointing to a commit that merges between the pull request and the
# upstream branch (GITHUB_HEAD_REF).
# upstream branch (GITHUB_BASE_REF).

set -euo pipefail

UPSTREAM_BRANCH="${GITHUB_HEAD_REF:-master}"
UPSTREAM_BRANCH="${GITHUB_BASE_REF:-master}"
echo "Checking for incompatible API changes relative to ${UPSTREAM_BRANCH}..."

MASTER_CLONE_DIR="$(mktemp -d)"
Expand Down
9 changes: 5 additions & 4 deletions internal/runtests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -71,9 +71,10 @@ echo "Ensuring that there are no dependencies not listed in ./internal/alldeps..

# For pull requests, check if there are undeclared incompatible API changes.
# Skip this if we're already going to fail since it is expensive.
if ${result} -eq 0 ]] && [[ ! -z "${GITHUB_HEAD_REF:-x}" ]]; then
echo
./internal/check_api_change.sh || result=1;
fi
# CURRENTLY BROKEN
# if [[ ${result} -eq 0 ]] && [[ ! -z "${GITHUB_HEAD_REF:-x}" ]]; then
# echo
# ./internal/check_api_change.sh || result=1;
# fi

exit $result

0 comments on commit 3b6fb78

Please sign in to comment.