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.

Sign up for GitHub

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

Jump to bottom

remove various files related to old wheel building mechanism #665

Merged
merged 2 commits into from
Dec 13, 2023
Merged

remove various files related to old wheel building mechanism #665

merged 2 commits into from
Dec 13, 2023

Conversation

grlee77
Copy link
Contributor

@grlee77 grlee77 commented Dec 13, 2023

closes #655

@grlee77
remove unused Docker images and dockcross-related files/scripts
56a3281 
we are now using standard RAPIDS containers to build wheels on CI rather than use a custom local process
@grlee77
remove docker folder with files used by run-dist and deprecated docum…
6e6319d 
…entation builds
@grlee77 grlee77 added improvement Improves an existing functionality non-breaking Introduces a non-breaking change maintenance labels Dec 13, 2023
@grlee77 grlee77 added this to the v24.02.00 milestone Dec 13, 2023
@grlee77 grlee77 requested a review from a team as a code owner December 13, 2023 16:46
gigony
gigony approved these changes Dec 13, 2023
View reviewed changes
Copy link
Contributor

@gigony gigony left a comment

Choose a reason for hiding this comment

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

Thanks @grlee77 ! Looks good to me!

@ajschmidt8 ajschmidt8 mentioned this pull request Dec 13, 2023
Merged
raydouglass pushed a commit that referenced this pull request Dec 13, 2023
@ajschmidt8
Update CODEOWNERS (#669)
ecdf36a 
This PR updates the `CODEOWNERS` file to ensure that the Ops team doesn't get tagged for files that we're not concerned with.

This should prevent PRs like the following from soliciting us for approvals:

- #665
- #666

[skip ci]

Authors:
   - AJ Schmidt (https://github.com/ajschmidt8)

Approvers:
   - Ray Douglass (https://github.com/raydouglass)
@grlee77
Copy link
Contributor Author

grlee77 commented Dec 13, 2023

/merge

@rapids-bot rapids-bot bot merged commit 45cc425 into rapidsai:branch-24.02 Dec 13, 2023
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ajschmidt8 ajschmidt8 ajschmidt8 approved these changes

@gigony gigony gigony approved these changes

Assignees
No one assigned
Labels
improvement Improves an existing functionality maintenance non-breaking Introduces a non-breaking change
Projects
cucim
Status: Done
Milestone
v24.02.00
Development

Successfully merging this pull request may close these issues.

Cleanup old build files
3 participants
@grlee77 @gigony @ajschmidt8