Skip to content

[UR][CI] Fix ur-build-hw workflow working-directory failure#22638

Merged
opensource-krzysztof merged 1 commit into
intel:syclfrom
opensource-krzysztof:fix-ur-build-hw-working-directory
Jul 16, 2026
Merged

[UR][CI] Fix ur-build-hw workflow working-directory failure#22638
opensource-krzysztof merged 1 commit into
intel:syclfrom
opensource-krzysztof:fix-ur-build-hw-working-directory

Conversation

@opensource-krzysztof

Copy link
Copy Markdown
Contributor

Remove working-directory: ./unified-runtime from the pip install step as the directory may not exist in merge commits. Use full relative paths to requirements files instead.

@opensource-krzysztof
opensource-krzysztof marked this pull request as ready for review July 15, 2026 10:50
@opensource-krzysztof
opensource-krzysztof requested a review from a team as a code owner July 15, 2026 10:50
@opensource-krzysztof
opensource-krzysztof force-pushed the fix-ur-build-hw-working-directory branch 3 times, most recently from 12e574b to b6ecf9c Compare July 15, 2026 11:29
Comment thread .github/workflows/ur-build-hw.yml Outdated
- Remove working-directory: ./unified-runtime from the pip install step
  and use full relative paths to requirements files instead.
- Add workspace cleanup step before checkout to remove stale files from
  previous runs that prevent git checkout from switching cleanly.
- Add explicit ref to checkout action to ensure correct ref is used.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@opensource-krzysztof
opensource-krzysztof force-pushed the fix-ur-build-hw-working-directory branch from b6ecf9c to 74cdc9b Compare July 15, 2026 12:15

@sarnex sarnex left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm but leaving to ur team to more in-depth review

@opensource-krzysztof
opensource-krzysztof merged commit d0f8f84 into intel:sycl Jul 16, 2026
40 of 41 checks passed
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.

4 participants