Skip to content

[rfile] Move RFile interface out of Experimental - #23637

Open
silverweed wants to merge 3 commits into
root-project:masterfrom
silverweed:rfile_out_of_exp
Open

silverweed wants to merge 3 commits into
root-project:masterfrom
silverweed:rfile_out_of_exp

Conversation

@silverweed

Copy link
Copy Markdown
Contributor

In preparation for 6.42. To be seen whether we want to move RNTupleWriter::Append(RFile) out of experimental or not

@silverweed silverweed self-assigned this Oct 7, 2026
@github-actions

github-actions Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Test Results

    24 files      24 suites   3d 17h 6m 44s ⏱️
 3 868 tests  3 846 ✅ 0 💤 22 ❌
76 293 runs  76 244 ✅ 0 💤 49 ❌

For more details on these failures, see this check.

Results for commit 0a4b620.

♻️ This comment has been updated with latest results.

@jblomer

jblomer commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

I think this can proceed.

@silverweed
silverweed force-pushed the rfile_out_of_exp branch 2 times, most recently from db7c770 to 5a40f96 Compare October 20, 2026 14:47
@silverweed
silverweed marked this pull request as ready for review October 20, 2026 14:50

@jblomer jblomer left a comment •

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.

LGTM. (maybe let the CI rerun)

@silverweed silverweed added the clean build Ask CI to do non-incremental build on PR label Oct 21, 2026
@silverweed silverweed closed this Oct 21, 2026
@silverweed silverweed reopened this Oct 21, 2026
@silverweed
silverweed requested a review from bellenot as a code owner October 21, 2026 08:55
@ferdymercury ferdymercury added this to the 6.42.00 milestone Oct 21, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clean build Ask CI to do non-incremental build on PR in:I/O

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants