Skip to content

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

Closed
ErichDonGubler wants to merge 10 commits intogfx-rs:trunkfrom
erichdongubler-mozilla:erichdongubler/push-kkxvvpmsqnqm
Closed

test(naga): add coverage for diagnostic directive parsing#6457
ErichDonGubler wants to merge 10 commits intogfx-rs:trunkfrom
erichdongubler-mozilla:erichdongubler/push-kkxvvpmsqnqm

Conversation

@ErichDonGubler
Copy link
Copy Markdown
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 Translation to Naga IR 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
@ErichDonGubler ErichDonGubler force-pushed the erichdongubler/push-kkxvvpmsqnqm branch from 3389296 to 0c5f071 Compare November 5, 2024 19:12
This resolves ambiguity that will be introduced in the subsequent commit
that adds `impl From<ConflictingDiagnosticRuleError> for naga::Error`.
@ErichDonGubler ErichDonGubler force-pushed the erichdongubler/push-kkxvvpmsqnqm branch from 0c5f071 to 3c78456 Compare November 8, 2024 02:29
@ErichDonGubler ErichDonGubler force-pushed the erichdongubler/push-kkxvvpmsqnqm branch from 3c78456 to ed4127d Compare November 8, 2024 15:58
@ErichDonGubler ErichDonGubler deleted the erichdongubler/push-kkxvvpmsqnqm branch November 8, 2024 19:10
@ErichDonGubler ErichDonGubler restored the erichdongubler/push-kkxvvpmsqnqm branch November 12, 2024 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: naga front-end Translation to Naga IR area: tests Improvements or issues with our test suite lang: WGSL WebGPU Shading Language naga Shader Translator

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant