Removing dependency from docs build#44
Removing dependency from docs build#44acostadon wants to merge 4 commits intorapidsai:branch-24.08from acostadon:ci_add
Conversation
jameslamb
left a comment
jameslamb
left a comment
There was a problem hiding this comment.
Makes sense to me, as this repo is using already-published cugraph packages and doesn't have CI to build them.
But in that case, lines like this in CI scripts should be removed too:
Lines 20 to 21 in c9fcee5
Also... should there be a branch-24.10 branch created here, and should PRs like this one target that instead?
| label_checker: true | ||
| release_drafter: false | ||
| recently_updated: true | ||
| forward_merger: false |
There was a problem hiding this comment.
This ops-bot.yaml file will be necessary to get RAPIDS CI running on the repo.
I think some additional manual steps will need to be done by ops to enable it https://github.com/rapidsai/ops/issues/3424
| "cugraph-dgl=${RAPIDS_VERSION_MAJOR_MINOR}.*" \ | ||
| "libcugraph_etl=${RAPIDS_VERSION_MAJOR_MINOR}.*" \ | ||
| "pylibcugraphops=${RAPIDS_VERSION_MAJOR_MINOR}.*" \ | ||
| "pylibwholegraph=${RAPIDS_VERSION_MAJOR_MINOR}.*" \ |
There was a problem hiding this comment.
Adding these versions to be sure that building the docs from an older branch pulls in the correct versions of all the cugraph libraries, instead of whatever happens to be the latest versions at the moment you run the build.
Replaces #44. Trying a new PR that enables docs builds on PRs and branch builds. Authors: - James Lamb (https://github.com/jameslamb) - Brad Rees (https://github.com/BradReesWork) - Jake Awe (https://github.com/AyodeAwe) Approvers: - Don Acosta (https://github.com/acostadon) - Mike Sarahan (https://github.com/msarahan) URL: #46
2 participants
No description provided.