Skip to content

Conversation

ferdymercury
Copy link
Collaborator

This Pull request:

Changes or fixes:

see https://github.com/root-project/root/actions/runs/17157474273/job/48677881584?pr=19688

since TestObj.cpp creates same-named file in same directory.

@ferdymercury ferdymercury requested a review from dpiparo as a code owner August 25, 2025 12:36
@ferdymercury ferdymercury requested a review from linev August 25, 2025 12:36
Copy link
Member

@linev linev left a comment

Choose a reason for hiding this comment

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

Do you have example of such conflict?

File "test.root" created in dedicated directory for root/tree/array tests
and therefore should not conflict with any other tests

@ferdymercury
Copy link
Collaborator Author

Do you have example of such conflict?

https://github.com/root-project/root/actions/runs/17157474273/job/48677881584?pr=19688

File "test.root" created in dedicated directory for root/tree/array tests

TestObj.cpp creates same-named file in the same root/tree/array directory.

Copy link
Member

@linev linev left a comment

Choose a reason for hiding this comment

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

Now I see. Thanks to fixing it.

Copy link

Test Results

    21 files      21 suites   3d 11h 42m 30s ⏱️
 3 561 tests  3 424 ✅  0 💤 137 ❌
72 974 runs  72 820 ✅ 17 💤 137 ❌

For more details on these failures, see this check.

Results for commit a4c3cc4.

@linev linev merged commit e847494 into root-project:master Aug 26, 2025
23 of 26 checks passed
@ferdymercury ferdymercury deleted the patch-7 branch August 26, 2025 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants