docs: add missing doxygen comment for par_nosync_t::on()#9196
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughSummary by CodeRabbit
suggestion: WalkthroughA Doxygen comment was added to document Changespar_nosync_t Documentation
Assessment against linked issues
Out-of-scope changesNo out-of-scope changes detected. The modification is limited to adding documentation for the Comment |
|
/ok to test ba66046 |
This comment has been minimized.
This comment has been minimized.
|
/ok to test aa93378 |
This comment has been minimized.
This comment has been minimized.
|
/ok to test 97151a9 |
This comment has been minimized.
This comment has been minimized.
|
@Oxygen56 please don't merge |
|
/ok to test 490ec1a |
Fixes NVIDIA#5453 Adds the //! Sets the stream documentation comment that par_t::on() already has, fixing the documentation inconsistency between the two execution policy types.
Head branch was pushed to by a user without write access
|
@bernhardmgruber Sorry about that! I've cleaned up the branch — force-pushed a single clean commit without the merge commits. CI should run cleanly now. Thanks for the approval and your patience! |
😬 CI Workflow Results🟥 Finished in 2h 47m: Pass: 91%/116 | Total: 4d 06h | Max: 2h 28m | Hits: 26%/923873See results here. |
Summary
//!comment onthrust::cuda::par_nosync_t::on()methodpar_t::on()method exactlythrust::cuda::par_nosync.onmissing #5453Fixes #5453
Test Plan
par_t::on()documentation