Skip to content

[core] emit warning for types headers deprecated in ROOT 6.20 - #23257

Merged
guitargeek merged 1 commit into
root-project:masterfrom
ferdymercury:htypes
Sep 4, 2026
Merged

[core] emit warning for types headers deprecated in ROOT 6.20#23257
guitargeek merged 1 commit into
root-project:masterfrom
ferdymercury:htypes

Conversation

@ferdymercury

Copy link
Copy Markdown
Collaborator

No description provided.

@ferdymercury
ferdymercury marked this pull request as ready for review September 4, 2026 11:20
@silverweed

silverweed commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

These headers weren't giving warnings about their deprecation status, were they?
In that case, shouldn't we first have a couple of releases with the actual warning visible to users before removing them?

@ferdymercury

Copy link
Copy Markdown
Collaborator Author

with the actual warning visible to users before removing them?

Fixed now.

@silverweed silverweed left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@ferdymercury ferdymercury changed the title [core] rm types headers deprecated in ROOT 6.20 [core] emit warning for types headers deprecated in ROOT 6.20 Sep 4, 2026
@dpiparo dpiparo self-assigned this Sep 4, 2026
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 18h 27m 35s ⏱️
 3 870 tests  3 869 ✅ 0 💤 1 ❌
79 745 runs  79 744 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 5c0a25f.

@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

@guitargeek
guitargeek merged commit 483dd9d into root-project:master Sep 4, 2026
32 of 34 checks passed
@ferdymercury
ferdymercury deleted the htypes branch September 5, 2026 05:50
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.

5 participants