Skip to content

[ntuple] Remove dependency from TError.h #18411

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[ntuple] Remove dependency from TError.h #18411

wants to merge 2 commits into from

Conversation

ferdymercury
Copy link
Contributor

@ferdymercury ferdymercury commented Apr 15, 2025 •
edited
Loading

This Pull request:

Changes or fixes:

Fixes https://its.cern.ch/jira/browse/ROOT-10805

RNTupleMerger class I marked it as to-do since I did not know if you want to throw an exception or rather write to stderr ?

Checklist:

  • tested changes locally
  • updated the docs (if necessary)

@ferdymercury ferdymercury requested a review from silverweed April 15, 2025 08:28
@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 15, 2025 •
edited
Loading

Test Results

    17 files      17 suites   3d 23h 55m 5s ⏱️
 2 744 tests  2 744 ✅ 0 💤 0 ❌
45 035 runs  45 035 ✅ 0 💤 0 ❌

Results for commit 741171d.

♻️ This comment has been updated with latest results.

@silverweed silverweed requested review from hahnjo and enirolf April 15, 2025 12:23
@ferdymercury ferdymercury modified the milestones: 6.38.00, 6.36.00 Apr 22, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@silverweed silverweed Awaiting requested review from silverweed

@jblomer jblomer Awaiting requested review from jblomer jblomer is a code owner

@dpiparo dpiparo Awaiting requested review from dpiparo dpiparo is a code owner

@pcanal pcanal Awaiting requested review from pcanal pcanal is a code owner

@hahnjo hahnjo Awaiting requested review from hahnjo

@enirolf enirolf Awaiting requested review from enirolf

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
6.36.00
Development

Successfully merging this pull request may close these issues.
1 participant
@ferdymercury