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

Shader crate cargo features #13

Merged
merged 2 commits into from
Sep 22, 2024

Conversation

Firestar99
Copy link
Member

@Firestar99 Firestar99 commented Sep 20, 2024

Requires #12 to be merged first

Adds fields for default-features and features to SpirvBuilder [and] pipes those fields through to the cargo command that builds the shader crate.

Original PR in embark's repo

@Firestar99 Firestar99 force-pushed the shader-crate-cargo-features branch 2 times, most recently from 4f65bbd to 1b7740c Compare September 20, 2024 11:05
@Firestar99 Firestar99 marked this pull request as ready for review September 20, 2024 11:23
@Firestar99 Firestar99 mentioned this pull request Sep 20, 2024
Copy link
Collaborator

@LegNeato LegNeato left a comment

Choose a reason for hiding this comment

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

Looks good. Not sure if we have tests for this or not, it we can add after.

@Firestar99 Firestar99 force-pushed the shader-crate-cargo-features branch from 1b7740c to 0ae6c4b Compare September 22, 2024 11:20
@LegNeato LegNeato merged commit 4c9718f into Rust-GPU:main Sep 22, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants