Skip to content

Comments

[main] Add Rootcp tests for --replace with entire file#21061

Merged
silverweed merged 2 commits intoroot-project:masterfrom
silverweed:rootcp_replace_entire
Feb 10, 2026
Merged

[main] Add Rootcp tests for --replace with entire file#21061
silverweed merged 2 commits intoroot-project:masterfrom
silverweed:rootcp_replace_entire

Conversation

@silverweed
Copy link
Contributor

@silverweed silverweed commented Jan 29, 2026

Based on #21067 (only last commit relevant).

Adds a roottest for rootcp --replace a.root b.root (where both a.root and b.root exist).

Serves as a test for #7052, which can be closed.

@silverweed silverweed requested a review from pcanal January 29, 2026 08:26
@silverweed silverweed self-assigned this Jan 29, 2026
@github-actions
Copy link

github-actions bot commented Jan 29, 2026

Test Results

    22 files      22 suites   3d 23h 2m 23s ⏱️
 3 795 tests  3 795 ✅ 0 💤 0 ❌
76 311 runs  76 311 ✅ 0 💤 0 ❌

Results for commit e834f0f.

♻️ This comment has been updated with latest results.

@silverweed silverweed marked this pull request as draft January 29, 2026 12:29
@silverweed silverweed force-pushed the rootcp_replace_entire branch from 2406bea to dd25b26 Compare January 29, 2026 12:54
@silverweed silverweed requested a review from vepadulano January 29, 2026 12:55
@silverweed silverweed marked this pull request as ready for review January 29, 2026 12:55
@silverweed silverweed force-pushed the rootcp_replace_entire branch from dd25b26 to e062316 Compare February 4, 2026 12:32
@silverweed silverweed requested a review from pcanal February 9, 2026 09:51
Copy link
Member

@pcanal pcanal left a comment

Choose a reason for hiding this comment

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

LGTM.

@silverweed silverweed force-pushed the rootcp_replace_entire branch from 1f23b8b to e834f0f Compare February 9, 2026 10:28
@silverweed silverweed merged commit d31f6ec into root-project:master Feb 10, 2026
48 of 53 checks passed
@silverweed silverweed deleted the rootcp_replace_entire branch February 10, 2026 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants