Skip to content

Conversation

@ErichDonGubler
Copy link
Member

@ErichDonGubler ErichDonGubler commented Aug 28, 2024

Connections

I think this was regressed by #6150?

Description

See above.

Testing

It compiles now!

Checklist

  • Run cargo fmt.

  • Run cargo clippy. If applicable, add:

    • --target wasm32-unknown-unknown
    • --target wasm32-unknown-emscripten
  • Run cargo xtask test to run tests.

  • Add change to CHANGELOG.md. See simple instructions inside file.

    Probably not necessary? Might want to add on to the glutin CHANGELOG entry. 🤔

@ErichDonGubler ErichDonGubler added the backend: gles Issues with GLES or WebGL label Aug 28, 2024
@ErichDonGubler ErichDonGubler self-assigned this Aug 28, 2024
@ErichDonGubler
Copy link
Member Author

@MarijnS95 ☝🏻

@MarijnS95
Copy link
Contributor

Doesn't your CI catch this?

Seems like I squashed this into the wrong PR (https://github.com/gfx-rs/wgpu/pull/6152/files#diff-ac2020a9d8bd724543c3bf9ff45b699d1ae9b15a9f6d77c38a604c7ff73cb716R19) as I was rebasing this from a Windows machine, which isn't supported in #6150 yet but only starting with #6152.

@teoxoy teoxoy closed this in #6176 Aug 28, 2024
@ErichDonGubler ErichDonGubler deleted the erichdongubler/push-mwmukqrqrmmz branch August 28, 2024 12:09
@ErichDonGubler
Copy link
Member Author

@MarijnS95: I'm not sure why our CI doesn't catch it. @cwfitzgerald, is there a reason CI doesn't at least try to compile examples?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend: gles Issues with GLES or WebGL

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants