Skip to content

Derive Hash on BindGroupLayoutEntry#799

Merged
bors[bot] merged 1 commit intogfx-rs:masterfrom
kunalmohan:derive-hash
Jul 16, 2020
Merged

Derive Hash on BindGroupLayoutEntry#799
bors[bot] merged 1 commit intogfx-rs:masterfrom
kunalmohan:derive-hash

Conversation

@kunalmohan
Copy link
Copy Markdown
Contributor

Connections
Link to the issues addressed by this PR, or dependent PRs in other repositories

Description
Describe what problem this is solving, and how it's solved.
Required in Servo.

Testing
Explain how this change is tested.
Not tested.

Copy link
Copy Markdown
Contributor

@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 0 warnings, 66 errors.

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

@kvark
Copy link
Copy Markdown
Member

kvark commented Jul 16, 2020

@kunalmohan 66 errors in in 2 lines, that's pretty impressive!
bors r+

@bors
Copy link
Copy Markdown
Contributor

bors bot commented Jul 16, 2020

@bors bors bot merged commit 8273bb3 into gfx-rs:master Jul 16, 2020
@kunalmohan kunalmohan deleted the derive-hash branch July 16, 2020 14:32
kvark added a commit to kvark/wgpu that referenced this pull request Jun 3, 2021
799: Update naga to gfx-16 r=kvark a=kvark

Makes WGSL constants much more ergonomic to write, and fixes the issues we had with the water example.

Co-authored-by: Dzmitry Malyshau <kvarkus@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