Skip to content
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

test(naga): add coverage for diagnostic directive parsing #6457

Draft
wants to merge 10 commits into
base: trunk
Choose a base branch
from

Conversation

ErichDonGubler
Copy link
Member

Tests diagnostic parsing added in #6456.

Checklist

  • Run cargo fmt.
  • Run taplo format.
  • 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.

@ErichDonGubler ErichDonGubler added area: tests Improvements or issues with our test suite naga Shader Translator area: naga front-end lang: WGSL WebGPU Shading Language labels Oct 23, 2024
@ErichDonGubler ErichDonGubler self-assigned this Oct 23, 2024
@ErichDonGubler ErichDonGubler changed the title test(naga): add cvg. for diagnostic directive parsing test(naga): add coverage for diagnostic directive parsing Oct 23, 2024
@ErichDonGubler ErichDonGubler force-pushed the erichdongubler/push-kkxvvpmsqnqm branch 10 times, most recently from 562bd78 to c97fc71 Compare October 30, 2024 21:11
@ErichDonGubler ErichDonGubler force-pushed the erichdongubler/push-kkxvvpmsqnqm branch 10 times, most recently from 23406ca to 3389296 Compare November 5, 2024 14:51
This resolves ambiguity that will be introduced in the subsequent commit
that adds `impl From<ConflictingDiagnosticRuleError> for naga::Error`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: naga front-end area: tests Improvements or issues with our test suite lang: WGSL WebGPU Shading Language naga Shader Translator
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

1 participant