NVIDIA / NVIDIA/cudf

Consolidate third-party `dependencies.yaml` into top-level `dependencies.yaml`

Open
#20,712 0 comments 0 reactions 0 assignees View on GitHub
cudf.pandas feature request
Dominant language
C++
Stars
9.8k
Forks
1.1k
Avg merge
3d 6m
Merged PRs (30d)
278

Description

Currently cuDF has a separate `dependencies.yaml` for third-party integration testing

https://github.com/rapidsai/cudf/blob/29fcf27d02ec3538d15e1a079ad4072e28289e81/python/cudf/cudf_pandas_tests/third_party_integration_tests/dependencies.yaml#L1

While this is was likely reasonable when setting this up at the time, we have noticed automated build logic updates frequently miss this nested file. Sometimes it is still missed during review

To simplify the update process and allow automated build updates to rollout smoothly in cuDF, this content would best be consolidated in the top-level `dependencies.yaml`

https://github.com/rapidsai/cudf/blob/bd340629a73ac87b8153e2b0e49cf91c16a6b13d/dependencies.yaml#L1

This way all the logic lives in the one `dependencies.yaml` file that is seeing regular updates with automated build updates

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.