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

Rename Component.instances to Component._instances to prevent collisions #470

Merged
merged 2 commits into from
Sep 10, 2024

Conversation

elikoga
Copy link
Member

@elikoga elikoga commented Sep 3, 2024

Concern was raised in flyingcircusio/batou_ext#190

@elikoga elikoga requested a review from zagy as a code owner September 3, 2024 09:08
zagy
zagy previously approved these changes Sep 3, 2024
@elikoga elikoga force-pushed the rename-component-instances-class-variable branch from cbe191f to b3c3589 Compare September 3, 2024 09:39
@elikoga elikoga force-pushed the rename-component-instances-class-variable branch from b3c3589 to b51b761 Compare September 10, 2024 08:23
@zagy zagy merged commit c980406 into main Sep 10, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants