Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

chore!: Remove running_total measure type #9170

Closed
wants to merge 2 commits into from

Conversation

igorlukanin
Copy link
Member

@igorlukanin igorlukanin commented Feb 1, 2025

BREAKING CHANGE: running_total measure type has long been deprecated. Use the rolling_window parameter instead.

@igorlukanin igorlukanin changed the title Remove running_total measure type chore!: Remove running_total measure type Feb 1, 2025
@KSDaemon KSDaemon requested a review from a team as a code owner February 3, 2025 10:23
@igorlukanin
Copy link
Member Author

Wow! It looks like supporting running_total required just a tiny bit of code.

@KSDaemon
Copy link
Member

KSDaemon commented Feb 3, 2025

@igorlukanin If I not missed somewhere else )))

@igorlukanin igorlukanin marked this pull request as draft February 5, 2025 11:13
@igorlukanin
Copy link
Member Author

We'll revive this PR later, when we're ready for v1.3.0.

@igorlukanin igorlukanin closed this Feb 5, 2025
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.

2 participants