Skip to content

Conversation

ferdymercury
Copy link
Collaborator

Closes #19942

@ferdymercury ferdymercury marked this pull request as ready for review September 22, 2025 20:53
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.

Not opposed to this per se, but I will let code owners make a final decision

Copy link

Test Results

    21 files      21 suites   3d 16h 59m 44s ⏱️
 3 664 tests  3 662 ✅ 0 💤 2 ❌
75 132 runs  75 126 ✅ 0 💤 6 ❌

For more details on these failures, see this check.

Results for commit 806b568.

Copy link
Contributor

@guitargeek guitargeek left a comment

Choose a reason for hiding this comment

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

Thank you!

@guitargeek guitargeek merged commit 581544f into root-project:master Sep 23, 2025
22 of 26 checks passed
@guitargeek
Copy link
Contributor

Damn I'm sorry I didn't squash the commits! My bad, it didn't enter my mind that this simple addition could be more than one commit, so I didn't even check and only noticed when scrolling the page back up after merging 🙁

@ferdymercury ferdymercury deleted the patch-20 branch September 23, 2025 06:19
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.

allow setting name of TStatistic or RDataFrame.Stats
3 participants