Skip to content

[df] Document copying behaviour of VariationsFor - #22992

Merged
vepadulano merged 1 commit into
root-project:masterfrom
vepadulano:df-variationsfor-copies-doc
Aug 28, 2026
Merged

vepadulano merged 1 commit into
root-project:masterfrom
vepadulano:df-variationsfor-copies-doc

Conversation

@vepadulano

Copy link
Copy Markdown
Member

Addresses #22956 (review)

@vepadulano
vepadulano requested a review from hahnjo August 3, 2026 11:24
@vepadulano vepadulano self-assigned this Aug 3, 2026
@vepadulano
vepadulano requested a review from hageboeck as a code owner August 3, 2026 11:24
@vepadulano vepadulano added in:RDataFrame skip ci Skip the full builds on the actions runners skip code analysis Skip the code analysis CI steps for this PR, including verifying clang-formatting and running Ruff. labels Aug 3, 2026
@github-actions

github-actions Bot commented Aug 3, 2026 •

Copy link
Copy Markdown

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 4bb7282.

♻️ This comment has been updated with latest results.

Comment thread tree/dataframe/src/RDataFrame.cxx Resolved
@vepadulano
vepadulano force-pushed the df-variationsfor-copies-doc branch from 7e389ad to 4bb7282 Compare August 6, 2026 13:15
@vepadulano
vepadulano requested a review from hahnjo August 6, 2026 13:15

@hageboeck hageboeck left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, possibly with an opportunity to shorten the first sentence a bit.

Comment thread tree/dataframe/inc/ROOT/RDFHelpers.hxx Resolved
@vepadulano
vepadulano force-pushed the df-variationsfor-copies-doc branch from 4bb7282 to d6ab7ce Compare August 28, 2026 13:08
@vepadulano
vepadulano merged commit b84bdad into root-project:master Aug 28, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in:RDataFrame skip ci Skip the full builds on the actions runners skip code analysis Skip the code analysis CI steps for this PR, including verifying clang-formatting and running Ruff.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants