Skip to content

Remove pod add read_spirv#280

Merged
bors[bot] merged 1 commit intogfx-rs:masterfrom
rukai:cleanup_examples
Aug 12, 2019
Merged

Remove pod add read_spirv#280
bors[bot] merged 1 commit intogfx-rs:masterfrom
rukai:cleanup_examples

Conversation

@rukai
Copy link
Copy Markdown
Contributor

@rukai rukai commented Aug 12, 2019

Looks like Pod is no longer used by wgpu-rs so I removed it.
read_spirv is needed in wgpu-rs now so I added it.

Copy link
Copy Markdown
Member

@kvark kvark left a comment

Choose a reason for hiding this comment

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

bors r+

bors Bot added a commit that referenced this pull request Aug 12, 2019
280: Remove pod add read_spirv r=kvark a=rukai

Looks like Pod is no longer used by wgpu-rs so I removed it.
read_spirv is needed in wgpu-rs now so I added it.

Co-authored-by: Rukai <rubickent@gmail.com>
@bors
Copy link
Copy Markdown
Contributor

bors Bot commented Aug 12, 2019

Build succeeded

@bors bors Bot merged commit 5763285 into gfx-rs:master Aug 12, 2019
bors Bot added a commit to gfx-rs/wgpu-rs that referenced this pull request Aug 12, 2019
61: Cleanup examples by using wgpu::read_spirv instead of manually creating a Vec<u32> r=kvark a=rukai

Wait for me to update the git reference once gfx-rs/wgpu#280 is merged.

Co-authored-by: Rukai <rubickent@gmail.com>
kvark pushed a commit to kvark/wgpu that referenced this pull request Jun 3, 2021
61: Cleanup examples by using wgpu::read_spirv instead of manually creating a Vec<u32> r=kvark a=rukai

Wait for me to update the git reference once gfx-rs#280 is merged.

Co-authored-by: Rukai <rubickent@gmail.com>
kvark pushed a commit to kvark/wgpu that referenced this pull request Jun 3, 2021
280: Add required wasm-bindgen patches to readme r=grovesNL a=rukai

Fixes gfx-rs/wgpu-rs#279

Co-authored-by: Rukai <rubickent@gmail.com>
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.

2 participants