Skip to content

Conversation

@KornevNikita
Copy link
Contributor

Ping gatekeepers only in PRs targeting the sycl branch. Also set default repo.

Ping gatekeepers only in PRs targeting the sycl branch.
Also set default repo.
@KornevNikita KornevNikita requested a review from a team as a code owner November 19, 2025 12:18
@KornevNikita
Copy link
Contributor Author

@AlexeySachkov FYI

# Skip PRs that don't target the sycl branch
pr_base=$(gh pr view $pr --json baseRefName -q .baseRefName)
if [ "$pr_base" != "sycl" ]; then
continue
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't want to do this for sycl-rel?

Copy link
Contributor

Choose a reason for hiding this comment

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

We don't want to do this for sycl-rel?

We have codeowners file reset on the sycl-rel branch so that I own everything and as such I'm aware of all the PRs.

I think that it is better to limit the number of gatekeepers involved in that, at least for now

@github-actions
Copy link
Contributor

@intel/llvm-gatekeepers please consider merging

@KornevNikita
Copy link
Contributor Author

@intel/llvm-gatekeepers could you please merge

@AlexeySachkov AlexeySachkov merged commit 635e6fa into intel:sycl Nov 20, 2025
30 checks passed
@AlexeySachkov AlexeySachkov deleted the upd-check-prs-wf branch November 20, 2025 14:57
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