Skip to content

Conversation

@baozaolaoba-top
Copy link
Contributor

Maintain a consistent usage pattern, especially when destroying entities.

Maintain a consistent usage pattern, especially when destroying
entities.
@Multirious Multirious mentioned this pull request May 7, 2025
5 tasks
@Multirious Multirious self-requested a review May 7, 2025 07:22
@Multirious
Copy link
Owner

Could you resolve the conflicts? Thanks.

Mainly for scenarios where entities may not exist.
@baozaolaoba-top
Copy link
Contributor Author

@Multirious please review

@baozaolaoba-top
Copy link
Contributor Author

@Multirious This is update to v0.9.1. Please let me know if any adjustments are needed. Thanks.

@baozaolaoba-top
Copy link
Contributor Author

@Rabbival Could you review this PR?

@Rabbival
Copy link
Collaborator

@Rabbival Could you review this PR?

That's a lot of line changes and I don't have context. I will read and let you know my thoughts, but it would be helpful if you summerized your changes with such a big PR

@Rabbival
Copy link
Collaborator

Rabbival commented Aug 18, 2025

Update: I found out (using a diff checker) that github's diff algorithm was being funny. The only new thing in your function is try_insert for commands instead of insert

@Rabbival Rabbival merged commit c587577 into Multirious:main Aug 18, 2025
3 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.

3 participants