Add no_std support to bevy_window#17031
Conversation
| "Please fix compiler errors in output above for bevy_state no_std compatibility.", | ||
| )); | ||
|
|
||
| commands.push(PreparedCommand::new::<Self>( |
There was a problem hiding this comment.
We should probably start deduplicating this logic soon.
There was a problem hiding this comment.
Yeah @BD103 and I had a chat about this earlier today. I like the idea of moving this out into a toml, csv or some other "plain data" file format.
There was a problem hiding this comment.
I also think we're rapidly reaching a critical mass of functionality where we could replace this CI task with just an example crate instead.
There was a problem hiding this comment.
It would be nice if this could be rolled up into a loop, but the macro requires a string literal, which means no format string.
TimJentzsch
left a comment
There was a problem hiding this comment.
I like the improved features documentation!
Head branch was pushed to by a user without write access
|
@alice-i-cecile just ran |
# Objective - Contributes to bevyengine#15460 ## Solution - Added the following features: - `std` (default) - `bevy_reflect` (default) - `libm` ## Testing - CI ## Notes - `bevy_reflect` was previously always enabled, which isn't how most other crates handle reflection. I've brought this in line with how most crates gate `bevy_reflect`. This is where the majority of the changes come from in this PR. --------- Co-authored-by: Alice Cecile <alice.i.cecile@gmail.com>
Objective
no_stdBevy #15460Solution
std(default)bevy_reflect(default)libmTesting
Notes
bevy_reflectwas previously always enabled, which isn't how most other crates handle reflection. I've brought this in line with how most crates gatebevy_reflect. This is where the majority of the changes come from in this PR.