Skip to content

[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

Merged
merged 10 commits into from
Feb 5, 2022

Conversation

alexbatashev
Copy link
Contributor

@alexbatashev alexbatashev commented Jan 11, 2022

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.

@alexbatashev alexbatashev marked this pull request as ready for review February 3, 2022 13:14
@alexbatashev alexbatashev requested a review from bader as a code owner February 3, 2022 13:14
Copy link
Contributor

@bader bader left a 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?

@alexbatashev
Copy link
Contributor Author

@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.

@bader
Copy link
Contributor

bader commented Feb 3, 2022

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?

@alexbatashev
Copy link
Contributor Author

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.

Copy link
Contributor

@bader bader left a comment

Choose a reason for hiding this comment

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

Fix pre-commit.

@alexbatashev alexbatashev marked this pull request as draft February 4, 2022 10:51
@alexbatashev alexbatashev marked this pull request as ready for review February 4, 2022 19:02
@alexbatashev
Copy link
Contributor Author

@bader bader merged commit 4447c2e into intel:sycl Feb 5, 2022
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.

2 participants