Add versionadded directives to all public API functions#7215
Merged
bernhardmgruber merged 6 commits intoNVIDIA:mainfrom Jan 17, 2026
Merged
Add versionadded directives to all public API functions#7215bernhardmgruber merged 6 commits intoNVIDIA:mainfrom
versionadded directives to all public API functions#7215bernhardmgruber merged 6 commits intoNVIDIA:mainfrom
Conversation
Add Sphinx `.. versionadded::` directives to all Thrust API functions that appear in the public documentation. This helps users understand which version of CCCL introduced each API. - Legacy APIs (pre-CCCL) are marked as versionadded:: 2.2.0 - reduce_into functions: 3.2.0 - transform_n and transform_if_n functions: 3.1.0 - inclusive_scan with init parameter: 2.6.0 Use `\verbatim embed:rst:leading-asterisk` with `/* */` style comments to ensure directives render correctly in Sphinx documentation. The `//!` comment style does not work with embed:rst as Doxygen includes the comment prefixes in the verbatim content.
Contributor
Contributor
bernhardmgruber
left a comment
bernhardmgruber
left a comment
There was a problem hiding this comment.
This is great, thank you!
Contributor
|
pre-commit.ci autofix |
Co-authored-by: Bernhard Manfred Gruber <bernhardmgruber@gmail.com>
Member
|
@cliffburdick this is awesome, how did you scrape the historical data without crazy tools like #6257 (comment)? |
Contributor
Author
Claude did 99% of the work and I verified it as much as I could. It used a combination of git blame and some rules I gave it for Thrust. |
Contributor
Author
It looks like clang format is still failing. Should I do this on my end? |
Contributor
|
pre-commit.ci autofix |
Contributor
|
/ok to test bb3e2c7 |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
cliffburdick
force-pushed
the
cliffburdick/thrust_api_version
branch
from
bb3e2c7 to
f101248
Compare
Contributor
|
pre-commit.ci autofix |
bernhardmgruber
approved these changes
Jan 17, 2026
Contributor
|
/ok to test 724d2d7 |
This comment has been minimized.
This comment has been minimized.
Contributor
🥳 CI Workflow Results🟩 Finished in 21h 49m: Pass: 100%/86 | Total: 1d 23h | Max: 3h 33m | Hits: 88%/125668See results here. |
shwina
pushed a commit
to shwina/cccl
that referenced
this pull request
Jan 18, 2026
Add Sphinx `.. versionadded::` directives to all Thrust API functions that appear in the public documentation. This helps users understand which version of CCCL introduced each API. - Legacy APIs (pre-CCCL) are marked as versionadded:: 2.2.0 - reduce_into functions: 3.2.0 - transform_n and transform_if_n functions: 3.1.0 - inclusive_scan with init parameter: 2.6.0 Use `\verbatim embed:rst:leading-asterisk` with `/* */` style comments to ensure directives render correctly in Sphinx documentation. The `//!` comment style does not work with embed:rst as Doxygen includes the comment prefixes in the verbatim content.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Add Sphinx
.. versionadded::directives to all Thrust API functions that appear in the public documentation. This helps users understand which version of CCCL introduced each API.Use
\verbatim embed:rst:leading-asteriskwith/* */style comments to ensure directives render correctly in Sphinx documentation. The//!comment style does not work with embed:rst as Doxygen includes the comment prefixes in the verbatim content.Verified correctness with local doc generation. New look: