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

Added AstcBlock and AstcChannel to the forwarded wgpu types. #14410

Merged
merged 1 commit into from
Jul 22, 2024

Conversation

Earthmark
Copy link
Contributor

@Earthmark Earthmark commented Jul 21, 2024

Currently TextureFormat::Astc can't be programmatically constructed without importing wgpu in addition to bevy.

Objective

Allow programmatic construction of TextureFormat::Astc with no additional imports required.

Solution

Exported the two component enums AstcBlock and AstcChannel used in TextureFormat::Astc construction.

Testing

I did not test this, the change seemed pretty safe. :)

Currently ASTC formats can't be programmatically constructed without importing wgpu in addition to bevy.
@alice-i-cecile alice-i-cecile added D-Trivial Nice and easy! A great choice to get started with Bevy A-Rendering Drawing game state to the screen S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it C-Usability A simple quality-of-life change that makes Bevy easier to use labels Jul 22, 2024
@alice-i-cecile
Copy link
Member

I wish we had explicit guidance on what types to forward, but the motivation here is solid.

@alice-i-cecile alice-i-cecile added this pull request to the merge queue Jul 22, 2024
Merged via the queue into bevyengine:main with commit 6928402 Jul 22, 2024
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Rendering Drawing game state to the screen C-Usability A simple quality-of-life change that makes Bevy easier to use D-Trivial Nice and easy! A great choice to get started with Bevy S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants