-
Notifications
You must be signed in to change notification settings - Fork 790
[CI] Automatically merge HEAD of sycl branch into jobs #5281
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Look okay, but I don't see new changes in pre-commit logs. Is that expected?
@bader yes, this action will be packed to docker image and delivered to all existing jobs with the new container. You can see the previous commit, where I deliberately get this action from repository and use the version from this PR. |
It looks like job the previous commit pulls all branches rather than a single target branch. Is it expected? |
Yes, that’s how github’s checkout action does it. But that’s fine since we keep a full clone locally. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix pre-commit.
Here's logs with new logic: https://github.com/intel/llvm/runs/5069696547?check_suite_focus=true |
Try to merge sycl branch into current git state on each test pipeline run. This is done to make sure the latest compiler is always built for testing changes.