rapidsai / rapids-cmake

https://docs.rapids.ai/api/rapids-cmake/stable/
Apache License 2.0
27 stars 45 forks source link

clarify language around how FetchContent is used in RAPIDS.cmake #590

Closed jameslamb closed 4 months ago

jameslamb commented 4 months ago

Description

Reading through, https://docs.rapids.ai/api/rapids-cmake/stable/basics/, I found this sentence a bit confusing:

The rapids-cmake module is designed to be acquired via CMake’s Fetch Content into your project.

Because the code block immediately below it does not actually call FetchContent() and friends.

image

This PR proposes a change to that language which I think might make it clearer that RAPIDS.cmake is itself using FetchContent().

Checklist

robertmaynard commented 4 months ago

/merge