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

Add no_std support to bevy_input #16995

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

bushrat011899
Copy link
Contributor

Objective

Solution

  • Added the following features:
    • std (default)
  • Fixed an existing issue where bevy_reflect wasn't properly feature gated.

Testing

  • CI

Notes

  • There were some minor issues with bevy_math and bevy_ecs noticed in this PR which I have also resolved here. I can split these out if desired, but I've left them here for now as they're very small changes and I don't consider this PR itself to be very controversial.

Also fixed some minor issues with `bevy_math` and `bevy_ecs`
@bushrat011899 bushrat011899 added C-Feature A new feature, making something new possible D-Trivial Nice and easy! A great choice to get started with Bevy A-Input Player input via keyboard, mouse, gamepad, and more X-Uncontroversial This work is generally agreed upon S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Dec 27, 2024
@bushrat011899 bushrat011899 changed the title Add no_std support to `bevy_input Add no_std support to bevy_input Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Input Player input via keyboard, mouse, gamepad, and more C-Feature A new feature, making something new possible D-Trivial Nice and easy! A great choice to get started with Bevy S-Needs-Review Needs reviewer attention (from anyone!) to move forward X-Uncontroversial This work is generally agreed upon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants