-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Split up bevy_core #16892
Copy link
Copy link
Closed
Labels
C-Code-QualityA section of code that is hard to understand or changeA section of code that is hard to understand or changeD-StraightforwardSimple bug fixes and API improvements, docs, test and examplesSimple bug fixes and API improvements, docs, test and examplesS-Ready-For-ImplementationThis issue is ready for an implementation PR. Go for it!This issue is ready for an implementation PR. Go for it!X-ContentiousThere are nontrivial implications that should be thought throughThere are nontrivial implications that should be thought through
Description
What problem does this solve or what need does it fill?
Title. This crate is basically a utilities crate, which we would like to avoid (bevy_utils is slowly being split up, for example). See here for its current API surface: https://docs.rs/bevy_core/latest/bevy_core/
What solution would you like?
We should:
- Move
FrameCountand its plugin tobevy_diagnosticsorbevy_window - Move other plugins to
bevy_app - Move everything else to
bevy_ecs
What alternative(s) have you considered?
Leave it as-is.
Additional context
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
C-Code-QualityA section of code that is hard to understand or changeA section of code that is hard to understand or changeD-StraightforwardSimple bug fixes and API improvements, docs, test and examplesSimple bug fixes and API improvements, docs, test and examplesS-Ready-For-ImplementationThis issue is ready for an implementation PR. Go for it!This issue is ready for an implementation PR. Go for it!X-ContentiousThere are nontrivial implications that should be thought throughThere are nontrivial implications that should be thought through