Skip to content

Conversation

@henrikmidtiby
Copy link
Contributor

Reviewer Checklist

  • The PR title is descriptive enough for the changelog, and the PR is labeled correctly
  • If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
  • If applicable: newly added functions and classes are tested

wiggle(alpha, self.n_wiggles) * self.rotation_angle,
about_point=self.get_rotate_about_point(),
)
return self
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Here the return value was changed to selfff

@henrikmidtiby henrikmidtiby marked this pull request as ready for review August 2, 2025 20:58
Copy link
Contributor

@chopan050 chopan050 left a comment

Choose a reason for hiding this comment

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

LGTM!

@github-project-automation github-project-automation bot moved this from 🆕 New to 👍 To be merged in Dev Board Aug 6, 2025
@chopan050 chopan050 enabled auto-merge (squash) August 6, 2025 13:43
@chopan050 chopan050 merged commit ded54e4 into ManimCommunity:main Aug 6, 2025
20 of 21 checks passed
@github-project-automation github-project-automation bot moved this from 👍 To be merged to ✅ Done in Dev Board Aug 6, 2025
@henrikmidtiby henrikmidtiby deleted the TypingIndication branch August 7, 2025 04:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants