Skip to content

Commit

Permalink
Revert "Temporarily disable docs errors. (#15265)" (#15269)
Browse files Browse the repository at this point in the history
This reverts part of commit c794ce4. This PR can be merged after the docs.rapids.ai issues are resolved. cc: @raydouglass @AyodeAwe

PR #15265 unblocked CI for cudf by disabling errors in the docs. However, it also included the diff from #15261 so that CI would pass. This reverts the docs build changes but leaves in the changes from #15261.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Jake Awe (https://github.com/AyodeAwe)

URL: #15269
  • Loading branch information
bdice authored Mar 11, 2024
1 parent c794ce4 commit c4f1a26
Showing 1 changed file with 1 addition and 13 deletions.
14 changes: 1 addition & 13 deletions ci/build_docs.sh
Original file line number Diff line number Diff line change
Expand Up @@ -34,11 +34,6 @@ rapids-mamba-retry install \

export RAPIDS_DOCS_DIR="$(mktemp -d)"

# TODO: Disable hard errors until the docs site is accessible (network problems)
EXITCODE=0
trap "EXITCODE=1" ERR
set +e

rapids-logger "Build CPP docs"
pushd cpp/doxygen
aws s3 cp s3://rapidsai-docs/librmm/html/${RAPIDS_VERSION_NUMBER}/rmm.tag . || echo "Failed to download rmm Doxygen tag"
Expand Down Expand Up @@ -71,11 +66,4 @@ if [[ "${RAPIDS_BUILD_TYPE}" != "pull-request" ]]; then
fi
popd

if [[ "${EXITCODE}" == "0" ]]; then
rapids-upload-docs
else
rapids-logger "Docs script had errors resulting in exit code $EXITCODE"
fi

# TODO: Disable hard errors until the docs site is accessible (network problems)
exit 0
rapids-upload-docs

0 comments on commit c4f1a26

Please sign in to comment.