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

Add get_loops_left() function to Tween #74454

Merged
merged 1 commit into from
Mar 25, 2023

Conversation

Pennycook
Copy link
Contributor

Implements godotengine/godot-proposals#5141.

Adds a new get_loops_left() function to Tween, allowing developers to reason about how many times a tweening sequence will repeat and whether to expect finished or loop_finished as the next signal.

@Pennycook
Copy link
Contributor Author

Rebased and force-pushed to restart the checks. They all failed last time, but the failures seemed unrelated to my changes.

@Pennycook
Copy link
Contributor Author

Tests are passing now, @KoBeWi.

scene/animation/tween.cpp Outdated Show resolved Hide resolved
scene/animation/tween.cpp Outdated Show resolved Hide resolved
Copy link
Member

@KoBeWi KoBeWi left a comment

Choose a reason for hiding this comment

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

Needs squashing.

Implements godotengine/godot-proposals#5141.

Adds a new get_loops_left() function to Tween, allowing developers to
reason about how many times a tweening sequence will repeat and whether
to expect finished or loop_finished as the next signal.

Co-authored-by: Tomek <[email protected]>
@Pennycook
Copy link
Contributor Author

@KoBeWi Squashed and pushed.

Is there anything else that needs to happen before this can be merged? I'm asking to understand the workflow (and make sure I didn't miss a step), not to try and rush this along.

@KoBeWi
Copy link
Member

KoBeWi commented Mar 22, 2023

No, it's ready to merge now.

@YuriSizov YuriSizov modified the milestones: 4.x, 4.1 Mar 25, 2023
@YuriSizov YuriSizov merged commit 82814f4 into godotengine:master Mar 25, 2023
@YuriSizov
Copy link
Contributor

Indeed, it is. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants