Skip to content

fix: buff the help menu for the wgpu-runner #254

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

Merged

Conversation

alphastrata
Copy link
Contributor

@alphastrata alphastrata commented May 21, 2025

Updates the help-menu on the wgpu-runner to avoid others making the mistake I made here.

Before:

Usage: example-runner-wgpu.exe [OPTIONS]

Options:
  -s, --shader <SHADER>       [default: Sky]
      --force-spirv-passthru
  -h, --help                  Print help

After:

Usage: example-runner-wgpu.exe [OPTIONS]

Options:
  -s, --shader <SHADER>       which shader to run [default: sky] [possible values: simplest, sky, compute, mouse]
      --force-spirv-passthru
  -h, --help                  Print help

closes #253

@Firestar99 Firestar99 enabled auto-merge May 21, 2025 10:04
@Firestar99 Firestar99 added this pull request to the merge queue May 21, 2025
auto-merge was automatically disabled May 21, 2025 10:19

Pull Request is not mergeable

@Firestar99 Firestar99 removed this pull request from the merge queue due to a manual request May 21, 2025
@Firestar99 Firestar99 force-pushed the bolster-clap-help-on-wgpu-runner branch from 814f19b to bbb826a Compare May 21, 2025 10:22
@Firestar99 Firestar99 enabled auto-merge May 21, 2025 10:22
@Firestar99
Copy link
Member

Firestar99 commented May 21, 2025

fixing the output filename collision warning on windows without breaking android is not trivial, let's leave it as is for now
#255

@Firestar99 Firestar99 disabled auto-merge May 21, 2025 10:25
@Firestar99 Firestar99 added this pull request to the merge queue May 21, 2025
Merged via the queue into Rust-GPU:main with commit 3b60bf9 May 21, 2025
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.

[BUG] Broken examples?
2 participants