Skip to content

Conversation

@nrspruit
Copy link
Contributor

@nrspruit nrspruit commented Aug 9, 2024

-pre-commit PR for oneapi-src/unified-runtime#1951

@sarnex
Copy link
Contributor

sarnex commented Aug 9, 2024

@nrspruit sorry i just happened to see notice this PR and I know it's a still draft, but since this looks like a DG2 specific change, DG2 testing isn't run in precommit besides trivial matrix tests. if you want to do full DG2 testing we need to run postcommit as part of PR testing, which you can do by creating a branch in intel/llvm prefixed with sycl-devops-pr/ and making a PR from there.

@nrspruit
Copy link
Contributor Author

nrspruit commented Aug 13, 2024

Thanks @sarnex

devops PR here:
#15054

@sarnex
Copy link
Contributor

sarnex commented Aug 13, 2024

Thanks, looks like postcommit is running there as expected!

-pre-commit PR for oneapi-src/unified-runtime#1951

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
@omarahmed1111
Copy link
Contributor

This will be included in this PR

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.

3 participants