Skip to content

Conversation

@aelovikov-intel
Copy link
Contributor

Aligns the task with its Linux counterpart.

…sent

Aligns the task with its Linux counterpart.
@aelovikov-intel
Copy link
Contributor Author

@bader
Copy link
Contributor

bader commented Jun 26, 2023

@aelovikov-intel, do you know why post-commit checks launched for the change in the pre-commit script?

@aelovikov-intel
Copy link
Contributor Author

aelovikov-intel commented Jun 26, 2023

@aelovikov-intel, do you know why post-commit checks launched for the change in the pre-commit script?

This is intentional - PR's source branch is in the origin/sycl-devops-pr/** format and is supposed to trigger post-commit on push. See #10002 's commit message for more details. I think I need to draft some docs/bkms regarding CI pre-commit testing too.

Edit: I just understood the question now. Yes, I think this can be improved slightly although it's not clear how to identify "changed" files on the push event.

@aelovikov-intel aelovikov-intel merged commit 92dbe1f into sycl Jun 26, 2023
@aelovikov-intel aelovikov-intel deleted the sycl-devops-pr/win-skip-lint branch June 26, 2023 21:05
Chenyang-L pushed a commit that referenced this pull request Jul 11, 2023
…sent (#10079)

Aligns the task with its Linux counterpart.
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