Skip to content

spirv_shader_passthrough must enable wgpu/spirv#15873

Merged
alice-i-cecile merged 2 commits intobevyengine:mainfrom
BenjaminBrienen:fix-feature-flag
Oct 13, 2024
Merged

spirv_shader_passthrough must enable wgpu/spirv#15873
alice-i-cecile merged 2 commits intobevyengine:mainfrom
BenjaminBrienen:fix-feature-flag

Conversation

@BenjaminBrienen
Copy link
Copy Markdown
Contributor

Objective

Fixes #15515

Solution

I went for the simplest solution because "format" in shader_format_spirv didn't sound directly related.

Testing

The command cargo b -p bevy --no-default-features -F spirv_shader_passthrough,x11 failed before, but works now.

@BenjaminBrienen BenjaminBrienen requested a review from BD103 October 12, 2024 20:55
@BenjaminBrienen BenjaminBrienen added C-Bug An unexpected or incorrect behavior D-Trivial Nice and easy! A great choice to get started with Bevy A-Reflection Runtime information about types S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Oct 12, 2024
@BenjaminBrienen BenjaminBrienen added this to the 0.15 milestone Oct 12, 2024
@IQuick143 IQuick143 added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Oct 12, 2024
@BenjaminBrienen BenjaminBrienen self-assigned this Oct 12, 2024
Copy link
Copy Markdown
Member

@BD103 BD103 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! :D

@alice-i-cecile alice-i-cecile added this pull request to the merge queue Oct 13, 2024
Merged via the queue into bevyengine:main with commit 37501e1 Oct 13, 2024
@BenjaminBrienen BenjaminBrienen deleted the fix-feature-flag branch October 29, 2024 21:56
@BenjaminBrienen BenjaminBrienen removed 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 Nov 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Reflection Runtime information about types C-Bug An unexpected or incorrect behavior D-Trivial Nice and easy! A great choice to get started with Bevy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

spirv_shader_passthrough must enable wgpu/spirv

6 participants