-
-
Notifications
You must be signed in to change notification settings - Fork 944
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
feat: Add multiFactory to SpawnComponent #3440
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall it looks good, just some minor comments
26244ea
to
36be5da
Compare
…ponents for each spawn tick. For backwards compatibility the single component factory is still supported and wrapped internally.
36be5da
to
e714216
Compare
The benchmark seems to be broken by itself ;) |
I did not close this! |
(I will rebase it though to see if my fix worked) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm! Thanks for your contribution.
You are welcome. Happy to help! |
add multiFactory to SpawnComponent to be able to create a list of components for each spawn tick. For backwards compatibility the single component factory is still supported and wrapped internally.
Description
This PR enhances the SpawnComponent to accept a multiFactory which adds a list of components per tick instead of just one.
Checklist
docs
and added dartdoc comments with///
.examples
ordocs
.Breaking Change?
Related Issues