Skip to content

Conversation

@MyreMylar
Copy link
Member

@MyreMylar MyreMylar commented Mar 19, 2023

To clean up the sprite documentation I've removed three classes that do nothing.

I've also removed OrderedUpdates from the sprite group code, it became effectively redundant after we dropped Python 3.5 support and all dictionaries became ordered. Ordered dictionaries were then made part of the Python specification in version 3.7 which is currently our lowest supported version.

Related to: #1941

To clean up the sprite documentation I've removed three classes that do nothing. I've also removed OrderedUpdates from the code, it became effectively redundant after we dropped Python 3.5 support and all dictionaries became ordered. Ordered dictionaries were then made part of the Python specification in version 3.7 which is currently our lowest supported version.
@MyreMylar MyreMylar requested a review from a team as a code owner March 19, 2023 18:24
@MyreMylar MyreMylar mentioned this pull request Mar 19, 2023
5 tasks
@yunline yunline added enhancement sprite pygame.sprite Code quality/robustness Code quality and resilience to changes Performance Related to the speed or resource usage of the project labels Mar 20, 2023
Copy link
Member

@ankith26 ankith26 left a comment

Choose a reason for hiding this comment

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

Nice cleanups, this indeed looks like the way forward, thanks for the PR 🍰

Copy link
Member

@oddbookworm oddbookworm left a comment

Choose a reason for hiding this comment

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

LGTM!

@oddbookworm oddbookworm added this to the 2.3 milestone Apr 12, 2023
@oddbookworm oddbookworm merged commit fcc4cc6 into main Apr 12, 2023
@MyreMylar MyreMylar deleted the sprite-deprecations branch May 29, 2023 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Code quality/robustness Code quality and resilience to changes enhancement Performance Related to the speed or resource usage of the project sprite pygame.sprite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants