Skip to content

Remove occurences of legacy-style naming#19551

Merged
vepadulano merged 2 commits intoroot-project:masterfrom
vepadulano:rdf-remove-old-headers
Aug 7, 2025
Merged

Remove occurences of legacy-style naming#19551
vepadulano merged 2 commits intoroot-project:masterfrom
vepadulano:rdf-remove-old-headers

Conversation

@vepadulano
Copy link
Member

@vepadulano vepadulano commented Aug 6, 2025

  • Remove two old headers which were not included in the library anyway
  • Change one occurrence of the old spelling in a tutorial
  • Remaining old TDS namespace and keyword occurrences

Copy link
Contributor

@martamaja10 martamaja10 left a comment

Choose a reason for hiding this comment

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

Nice! Thanks :)

@vepadulano vepadulano added the clean build Ask CI to do non-incremental build on PR label Aug 6, 2025
@vepadulano vepadulano closed this Aug 6, 2025
@vepadulano vepadulano reopened this Aug 6, 2025
@couet couet removed their request for review August 6, 2025 08:17
@github-actions
Copy link

github-actions bot commented Aug 6, 2025

Test Results

    21 files      21 suites   3d 9h 46m 47s ⏱️
 3 247 tests  3 247 ✅ 0 💤 0 ❌
66 466 runs  66 466 ✅ 0 💤 0 ❌

Results for commit 03df8fa.

♻️ This comment has been updated with latest results.

* Remove two old headers which were not included in the library anyway
* Change one occurrence of the old spelling in a tutorial
@vepadulano vepadulano force-pushed the rdf-remove-old-headers branch from 1c917ab to 03df8fa Compare August 6, 2025 11:11
@vepadulano vepadulano changed the title Remove remaining TDataFrame occurrences Remove occurences of legacy-style naming Aug 6, 2025
@vepadulano vepadulano merged commit 1ea62ff into root-project:master Aug 7, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clean build Ask CI to do non-incremental build on PR in:RDataFrame

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants