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

[Merged by Bors] - expose texture/image conversions as From/TryFrom #2175

Closed
wants to merge 3 commits into from

Conversation

mockersf
Copy link
Member

fixes #2169

Instead of having custom methods with reduced visibility, implement From<image::DynamicImage> for Texture and TryFrom<Texture> for image::DynamicImage

@alice-i-cecile alice-i-cecile added C-Enhancement A new feature A-Rendering Drawing game state to the screen labels May 16, 2021
mockersf and others added 3 commits May 18, 2021 01:48
Co-Authored-By: Nathan Ward <43621845+NathanSWard@users.noreply.github.com>
Co-authored-by: Nathan Ward <43621845+NathanSWard@users.noreply.github.com>
@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label May 19, 2021
@cart
Copy link
Member

cart commented Jun 8, 2021

Love it!

@cart
Copy link
Member

cart commented Jun 8, 2021

bors r+

bors bot pushed a commit that referenced this pull request Jun 8, 2021
fixes #2169 

Instead of having custom methods with reduced visibility, implement `From<image::DynamicImage> for Texture` and `TryFrom<Texture> for image::DynamicImage`
@bors bors bot changed the title expose texture/image conversions as From/TryFrom [Merged by Bors] - expose texture/image conversions as From/TryFrom Jun 8, 2021
@bors bors bot closed this Jun 8, 2021
ostwilkens pushed a commit to ostwilkens/bevy that referenced this pull request Jul 27, 2021
fixes bevyengine#2169 

Instead of having custom methods with reduced visibility, implement `From<image::DynamicImage> for Texture` and `TryFrom<Texture> for image::DynamicImage`
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-Enhancement A new feature 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.

image texture
4 participants