[FEA] Add tests to verify headers are standalone
Nobody has claimed this yet.
- Dominant language
- Cuda
- Stars
- 667
- Forks
- 120
- Avg merge
- 7d 5h
- Merged PRs (30d)
- 4
Description
Is your feature request related to a problem? Please describe.
After @PointKernel's excellent work in https://github.com/NVIDIA/cuCollections/pull/141 to ensure each cuco header is standalone, we should add some tests to ensure they stay that way.
Describe the solution you'd like
@allisonvacanti pointed out the fact that Thrust does this by simply including each header in an otherwise empty .cu file and ensuring it compiles. We should steal what they do.
See
CMake: https://github.com/NVIDIA/thrust/blob/main/cmake/ThrustHeaderTesting.cmake
Configured header: https://github.com/NVIDIA/thrust/blob/main/cmake/header_test.in
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the linked Thrust CMake header-testing implementation and its header_test.in template. Add equivalent tests for cuCollections so each cuco header is included in an otherwise empty .cu file and compiles successfully.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cmake, cpp
- Domain
- build-system, testing
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100