Skip to content

Add constraint to test that requires spirv support#7241

Merged
bob80905 merged 1 commit intomicrosoft:mainfrom
bob80905:Add_missing_spirv_requires_to_test
Mar 21, 2025
Merged

Add constraint to test that requires spirv support#7241
bob80905 merged 1 commit intomicrosoft:mainfrom
bob80905:Add_missing_spirv_requires_to_test

Conversation

@bob80905
Copy link
Copy Markdown
Collaborator

This PR adds a // REQUIRES: spirv line to the top of a test that uses spirv.
This prevents failures in dev environments that don't have spirv enabled.

@bob80905
Copy link
Copy Markdown
Collaborator Author

@s-perron
Want to bring this to your attention, I don't think it'll reduce coverage.

@bob80905 bob80905 enabled auto-merge (squash) March 21, 2025 00:42
@bob80905 bob80905 merged commit 60e6c76 into microsoft:main Mar 21, 2025
13 checks passed
@github-project-automation github-project-automation Bot moved this from New to Done in HLSL Roadmap Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants