Skip to content

[roottest] improve ntuple cmake #19582

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 11, 2025
Merged

Conversation

linev
Copy link
Member

@linev linev commented Aug 8, 2025

Skip root7 test - RNTuple always build
Properly use multiple fixtures, use longer fixtures names
Directly use multiple sources when build executables

Skip root7 test - RNTuple always build
Properly use multiple fixtures, use longer fixtures names
Directly use multiple sources when build executables
@linev linev requested a review from vepadulano August 8, 2025 10:52
@linev linev self-assigned this Aug 8, 2025
@linev linev requested a review from bellenot as a code owner August 8, 2025 10:52
@linev linev requested a review from dpiparo as a code owner August 8, 2025 10:52
Copy link
Member

@vepadulano vepadulano left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

github-actions bot commented Aug 8, 2025

Test Results

    21 files      21 suites   3d 7h 41m 17s ⏱️
 3 250 tests  3 247 ✅ 0 💤 3 ❌
66 537 runs  66 530 ✅ 0 💤 7 ❌

For more details on these failures, see this check.

Results for commit 17c5c3a.

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.

Thanks.

@linev linev merged commit 8fc9ae7 into root-project:master Aug 11, 2025
25 of 30 checks passed
@linev linev deleted the roottest_ntuple branch August 11, 2025 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants