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

Event scheduling #13439

Closed
wants to merge 5 commits into from
Closed

Event scheduling #13439

wants to merge 5 commits into from

Conversation

lee-orr
Copy link
Contributor

@lee-orr lee-orr commented May 20, 2024

Adds the ability to set up schedules that trigger when events are sent.

@alice-i-cecile alice-i-cecile added C-Enhancement A new feature A-ECS Entities, components, systems, and events X-Controversial There is active debate or serious implications around merging this PR S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged labels May 21, 2024
@alice-i-cecile
Copy link
Member

I'll want to see a strong argument for a) what role does this serve for users and b) why does this need to exist in bevy_ecs before I'm in favor of this. I would really, really like to keep our scheduling abstractions minimal.

@lee-orr
Copy link
Contributor Author

lee-orr commented May 24, 2024

This was brought up as part of other work (same-state transitions) - but I don't believe it is actually worthwhile at this point.

@lee-orr lee-orr closed this May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ECS Entities, components, systems, and events C-Enhancement A new feature S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged X-Controversial There is active debate or serious implications around merging this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants