Skip to content

Fix Doxygen @param entries in /include#21762

Merged
rapids-bot[bot] merged 6 commits intorapidsai:mainfrom
vuule:docs-param-fixes-include
Mar 16, 2026
Merged

Fix Doxygen @param entries in /include#21762
rapids-bot[bot] merged 6 commits intorapidsai:mainfrom
vuule:docs-param-fixes-include

Conversation

@vuule
Copy link
Copy Markdown
Contributor

@vuule vuule commented Mar 12, 2026

Description

AI-assisted, human-verified, fixes for the Doxygen parameter documentation - limited to /include directory.

We often find these kinds of errors in new PRs. With AI code review we will probably stop merging new errors of this type. This PR aims to fix old error so we hopefully stay 100% accurate from now on.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Mar 12, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions github-actions Bot added the libcudf Affects libcudf (C++/CUDA) code. label Mar 12, 2026
@vuule vuule added doc Documentation improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Mar 12, 2026
@vuule vuule removed the improvement Improvement / enhancement to an existing function label Mar 12, 2026
@vuule
Copy link
Copy Markdown
Contributor Author

vuule commented Mar 12, 2026

/ok to test d01d060

@davidwendt
Copy link
Copy Markdown
Contributor

This looks good.
Just wanted to note here for reviewers that these are all detail declarations so they don't appear in our public docs.
Also, noticing that the bit.hpp header declares only detail functions but is not in a detail folder.

@vuule vuule marked this pull request as ready for review March 12, 2026 18:09
@vuule vuule requested a review from a team as a code owner March 12, 2026 18:09
@vuule vuule requested review from bdice and shrshi March 12, 2026 18:09
*
* @param p A \c pointer to the previously allocated memory.
* @note The second parameter is the number of objects previously allocated.
* @param cnt Number of objects that werepreviously allocated.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing a space in this sentence. I also wish this was count...

Suggested change
* @param cnt Number of objects that werepreviously allocated.
* @param cnt Number of objects that were previously allocated.

@vuule
Copy link
Copy Markdown
Contributor Author

vuule commented Mar 16, 2026

/ok to test 0e2c513

@vuule vuule added the 5 - Ready to Merge Testing and reviews complete, ready to merge label Mar 16, 2026
@vuule
Copy link
Copy Markdown
Contributor Author

vuule commented Mar 16, 2026

/merge

@rapids-bot rapids-bot Bot merged commit cfa6ef1 into rapidsai:main Mar 16, 2026
117 checks passed
@vuule vuule deleted the docs-param-fixes-include branch March 16, 2026 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5 - Ready to Merge Testing and reviews complete, ready to merge doc Documentation libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants