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

Remove get_ compatibility layer #3773

Closed
wants to merge 8 commits into from

Conversation

JasonGrace2282
Copy link
Member

This has been deprecated for some time now. Getting the attribute manually is now the supported way.

@JasonGrace2282 JasonGrace2282 added pr:deprecation Deprecation, or removal of deprecated code breaking changes This PR introduces breaking changes labels May 20, 2024
@JasonGrace2282 JasonGrace2282 added the needs discussion Things which needs to be discussed before implemented. label May 21, 2024
@JasonGrace2282 JasonGrace2282 marked this pull request as draft July 3, 2024 13:25
@JasonGrace2282
Copy link
Member Author

This PR seems like a good candidate for also changing the way getters/setters are in Manim - see the Discord developement thread

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking changes This PR introduces breaking changes needs discussion Things which needs to be discussed before implemented. pr:deprecation Deprecation, or removal of deprecated code
Projects
Status: Rejected
Development

Successfully merging this pull request may close these issues.

2 participants