Skip to content

Commit

Permalink
Allow skip link checks or warn link checks but not both (#2844)
Browse files Browse the repository at this point in the history
  • Loading branch information
nkammah authored Dec 1, 2023
1 parent d90acf5 commit cefde70
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 7 deletions.
6 changes: 2 additions & 4 deletions .buildkite/scripts/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -20,10 +20,8 @@ fi

if [[ "${BROKEN_LINKS}" == 'skiplinkcheck' ]]; then
broken_links_opt="--skiplinkcheck"
else
if [[ "${BROKEN_LINKS}" == 'warnlinkcheck' ]]; then
broken_links_opt="--warnlinkcheck"
fi
elif [[ "${BROKEN_LINKS}" == 'warnlinkcheck' ]]; then
broken_links_opt="--warnlinkcheck"
fi

if [[ "${BUILDKITE_BRANCH}" == "master" ]]; then
Expand Down
5 changes: 2 additions & 3 deletions .buildkite/scripts/build_pr.sh
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,8 @@ if [[ ${GITHUB_PR_COMMENT_VAR_REBUILD_OPT:="unset"} == "rebuild" ]];then
rebuild_opt=" --rebuild"
elif [[ ${GITHUB_PR_COMMENT_VAR_SKIP_OPT:="unset"} == "skiplinkcheck" ]];then
build_args+=" --skiplinkcheck"
if [[ ${GITHUB_PR_COMMENT_VAR_WARN_OPT:="unset"} == "warnlinkcheck" ]];then
build_args+=" --warnlinkcheck"
fi
elif [[ ${GITHUB_PR_COMMENT_VAR_WARN_OPT:="unset"} == "warnlinkcheck" ]];then
build_args+=" --warnlinkcheck"
fi

buildkite-agent \
Expand Down

0 comments on commit cefde70

Please sign in to comment.