Skip to content

Conversation

@bob80905
Copy link
Collaborator

@bob80905 bob80905 commented Oct 27, 2025

This PR just addresses some final concerns in #7749
It also adds a missing REQUIRES line that should've been added to a test that merged into the repo while the previous PR was in development.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 27, 2025

✅ With the latest revision this PR passed the C/C++ code formatter.

Copy link
Collaborator

@joaosaffran joaosaffran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, not really familiar with the codebase though

@bob80905 bob80905 merged commit b6c1914 into microsoft:main Oct 28, 2025
13 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in HLSL Roadmap Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants