Skip to content

[net] deprecate NetErrors.h implementation detail - #23389

Open
ferdymercury wants to merge 2 commits into
root-project:masterfrom
ferdymercury:bneterr
Open

ferdymercury wants to merge 2 commits into
root-project:masterfrom
ferdymercury:bneterr

Conversation

@ferdymercury

Copy link
Copy Markdown
Collaborator

only private .cxx files use this header
not a single function within ROOT returns or accepts as argument publicly the struct defined in this header, so no one can really utilize it.

@ferdymercury ferdymercury changed the title [net] deprecate NetErrors implementation detail [net] deprecate NetErrors.h implementation detail Sep 16, 2026
@ferdymercury ferdymercury added the skip code analysis Skip the code analysis CI steps for this PR, including verifying clang-formatting and running Ruff. label Sep 16, 2026
@ferdymercury ferdymercury reopened this Sep 16, 2026
@ferdymercury ferdymercury added this to the 6.42.00 milestone Sep 16, 2026
@github-actions

github-actions Bot commented Sep 16, 2026

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 20h 11m 30s ⏱️
 3 879 tests  3 878 ✅ 0 💤 1 ❌
80 057 runs  80 056 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 2dc01fb.

♻️ This comment has been updated with latest results.

Comment thread net/net/inc/NetErrors.h Outdated

@pcanal pcanal left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks.

Co-authored-by: Philippe Canal <pcanal@fnal.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip code analysis Skip the code analysis CI steps for this PR, including verifying clang-formatting and running Ruff.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants