Skip to content
This repository was archived by the owner on Jun 18, 2021. It is now read-only.

Update wgpu to the new render pipeline descriptor API#734

Merged
bors[bot] merged 1 commit intogfx-rs:masterfrom
kvark:render-api
Jan 30, 2021
Merged

Update wgpu to the new render pipeline descriptor API#734
bors[bot] merged 1 commit intogfx-rs:masterfrom
kvark:render-api

Conversation

@kvark
Copy link
Copy Markdown
Member

@kvark kvark commented Jan 29, 2021

MAJOR API BREAKING CHANGE
Gets us gfx-rs/wgpu#1176

Copy link
Copy Markdown

@monocodus monocodus bot left a comment

Choose a reason for hiding this comment

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

This is an autogenerated code review.

Checker summary (by rust_clippy):
The tool has found 1 warnings, 6 errors.

The .monocodus config not found in your repo. Default config is used.
Check config documentation here

Copy link
Copy Markdown

@monocodus monocodus bot left a comment

Choose a reason for hiding this comment

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

This is an autogenerated code review.

Checker summary (by rust_clippy):
The tool has found 1 warnings, 6 errors.

The .monocodus config not found in your repo. Default config is used.
Check config documentation here

Copy link
Copy Markdown
Collaborator

@grovesNL grovesNL left a comment

Choose a reason for hiding this comment

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

Looks great 👍 The examples look really nice with the new API!

@kvark
Copy link
Copy Markdown
Member Author

kvark commented Jan 30, 2021

Thank you for reviewing!
bors r=grovesNL

@bors
Copy link
Copy Markdown
Contributor

bors bot commented Jan 30, 2021

@bors bors bot merged commit c45fbce into gfx-rs:master Jan 30, 2021
@kvark kvark deleted the render-api branch January 30, 2021 05:27
PixelDust22 added a commit to PixelDust22/bevy that referenced this pull request Jan 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants