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-build-backend #4393

Merged
merged 65 commits into from
Jul 1, 2024

Conversation

KyleFromNVIDIA
Copy link
Contributor

@KyleFromNVIDIA KyleFromNVIDIA commented May 7, 2024

Contributes to rapidsai/build-planning#31
Contributes to rapidsai/dependency-file-generator#89

Proposes introducing rapids-build-backend as this project's build backend, to reduce the complexity of various CI/build scripts.

@KyleFromNVIDIA KyleFromNVIDIA requested review from a team as code owners May 7, 2024 19:18
@github-actions github-actions bot added the conda label May 8, 2024
@KyleFromNVIDIA KyleFromNVIDIA added improvement Improvement / enhancement to an existing function DO NOT MERGE Hold off on merging; see PR for details non-breaking Non-breaking change labels May 16, 2024
@jameslamb
Copy link
Member

The test failures look like this issue unrelated to this PR: #4496 (comment).

I believe this PR is now ready for review.

@jameslamb jameslamb removed the request for review from a team June 27, 2024 17:23
@jameslamb jameslamb self-assigned this Jul 1, 2024
@BradReesWork
Copy link
Member

LGTM from a doc codeowner perspective

@jameslamb
Copy link
Member

The other changes on branch-24.08 since this was last built don't look like they'd affect or be affected by these changes, so I'm going to merge this. Thanks everyone!

@jameslamb
Copy link
Member

/merge

@rapids-bot rapids-bot bot merged commit 538a2ce into rapidsai:branch-24.08 Jul 1, 2024
131 checks passed
naimnv pushed a commit to naimnv/cugraph-forked that referenced this pull request Jul 2, 2024
naimnv pushed a commit to naimnv/cugraph-forked that referenced this pull request Jul 2, 2024
rapids-bot bot pushed a commit to rapidsai/cugraph-gnn that referenced this pull request Oct 30, 2024
Another steps towards completing the work started in #53 

Fixes #15

Contributes to rapidsai/build-planning#111

Proposes changes to get CI running on pull requests for `cugraph-pyg` and `cugraph-dgl`

## Notes for Reviewers

Workflows for nightly builds and publishing nightly packages are intentionally not included here. See #58 (comment)

Notebook tests are intentionally not added here... they'll be added in the next PR.

Pulls in changes from these other upstream PRs that had not been ported over to this repo:

* rapidsai/cugraph#4690
* rapidsai/cugraph#4393

Authors:
  - James Lamb (https://github.com/jameslamb)
  - Alex Barghi (https://github.com/alexbarghi-nv)

Approvers:
  - Alex Barghi (https://github.com/alexbarghi-nv)
  - Bradley Dice (https://github.com/bdice)

URL: #59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci conda improvement Improvement / enhancement to an existing function non-breaking Non-breaking change python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants