-
-
Notifications
You must be signed in to change notification settings - Fork 21.1k
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
Fix the tween null check #45438
Fix the tween null check #45438
Conversation
@Calinou is this ok? ;D |
Code looks good to me (apart from the comments below), but I don't merge code pull requests on the main repository. You'll have to wait for Akien to take a look. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it'd be good to add a descriptive message to help the user troubleshoot the issue:
I force pushed a rebase to squash the commits into one and clarify the commit message. |
Thanks! |
Cherry-picked for 3.2.4. |
well this fixes the null check for tween for 4.0, the 3.2 code looks a little different so i dont know how to push both to the same master as this is my first time playing around with this :)
fixes #45399 error for version 4