Skip to content

Conversation

@jameslamb
Copy link
Member

Description

Contributes to rapidsai/build-planning#181

  • removes all uploads of conda packages and wheels to downloads.rapids.ai

Notes for Reviewers

How I identified changes

Looked for uses of the relevant gha-tools tools, as well as documentation about downloads.rapids.ai, being on the NVIDIA VPN, using S3, etc. like this:

git grep -i -E 's3|upload|downloads\.rapids|vpn'

How I tested this

See "How I tested this" on rapidsai/shared-workflows#364

@jameslamb jameslamb changed the title WIP: stop uploading packages to downloads.rapids.ai stop uploading packages to downloads.rapids.ai May 27, 2025
@jameslamb jameslamb marked this pull request as ready for review May 27, 2025 22:56
@jameslamb jameslamb requested a review from AjayThorve May 27, 2025 22:57
Copy link
Member

@AjayThorve AjayThorve left a comment

Choose a reason for hiding this comment

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

Lgtm

@jameslamb
Copy link
Member Author

/merge

@AjayThorve AjayThorve merged commit ca35dda into rapidsai:branch-0.14 May 28, 2025
11 checks passed
@jameslamb jameslamb mentioned this pull request May 28, 2025
AjayThorve pushed a commit that referenced this pull request May 29, 2025
Proposes adding a `CODEOWNERS` file to automatically assign reviews, and
using it to split reviewing responsibility as follows:

* CI changes: https://github.com/orgs/rapidsai/teams/ci-codeowners
* packaging changes:
https://github.com/orgs/rapidsai/teams/packaging-codeowners
* all other changes:
https://github.com/orgs/rapidsai/teams/jupyterlab-nvdashboard-write

The main goal of this change is to reduce the reviewing burden on this
library's main developers for RAPIDS-wide packaging changes like #236 .

---------

Co-authored-by: jakirkham <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants