Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use rapids-upload-docs script #13518

Merged
merged 2 commits into from
Jun 7, 2023
Merged

Conversation

AyodeAwe
Copy link
Contributor

@AyodeAwe AyodeAwe commented Jun 6, 2023

This PR updates the build_docs.sh script to use the new consolidatory rapids-upload-script shared script.

The shared script enables docs uploads to applicable S3 buckets for branch. nightly and PR builds.

@AyodeAwe AyodeAwe added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jun 6, 2023
@AyodeAwe AyodeAwe requested a review from a team as a code owner June 6, 2023 20:47
@github-actions github-actions bot added the ci label Jun 6, 2023
pushd cpp/doxygen
aws s3 cp s3://rapidsai-docs/librmm/${VERSION_NUMBER}/html/rmm.tag . || echo "Failed to download rmm Doxygen tag"
aws s3 cp s3://rapidsai-docs/librmm/${RAPIDS_VERSION_NUMBER}/html/rmm.tag . || echo "Failed to download rmm Doxygen tag"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We'll need to make a note of paths like these so that we can update them when we restructure the S3 bucket.

I will add this to our issue about the bucket restructure.

@ajschmidt8
Copy link
Member

/merge

@rapids-bot rapids-bot bot merged commit 8055c2d into rapidsai:branch-23.08 Jun 7, 2023
@ajschmidt8 ajschmidt8 mentioned this pull request Jun 8, 2023
@ajschmidt8
Copy link
Member

fixed a dask-cudf typo in #13536

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants