Skip to content

Conversation

@aelovikov-intel
Copy link
Contributor

I'm adding Chris and myself explicitly as we're expected to be the most involved people with that area (and I hope that Github will just happen to work in some nice way due to that), but I'm not limiting the "ownership" to just the two of us.

I'm adding Chris and myself explicitly as we're expected to be the most
involved people with that area (and I hope that Github will just happen
to work in some nice way due to that), but I'm not limiting the
"ownership" to just the two of us.
@aelovikov-intel aelovikov-intel requested a review from a team as a code owner September 17, 2025 19:25
Copy link
Contributor

@sarnex sarnex left a comment

Choose a reason for hiding this comment

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

no idea what github will do in this case but lgtm

@github-actions
Copy link
Contributor

@intel/llvm-gatekeepers please consider merging

Copy link
Contributor

@stdale-intel stdale-intel left a comment

Choose a reason for hiding this comment

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

LGTM

@aelovikov-intel aelovikov-intel merged commit 5bb03ae into intel:sycl Sep 18, 2025
3 checks passed
@aelovikov-intel aelovikov-intel deleted the rtc-codeowners branch September 18, 2025 15:33
@aelovikov-intel
Copy link
Contributor Author

no idea what github will do in this case but lgtm

Looks like it did the right thing (#20115):

image

and

image

@sarnex
Copy link
Contributor

sarnex commented Sep 18, 2025

wow thats a first

YixingZhang007 pushed a commit to YixingZhang007/llvm that referenced this pull request Sep 22, 2025
I'm adding Chris and myself explicitly as we're expected to be the most
involved people with that area (and I hope that Github will just happen
to work in some nice way due to that), but I'm not limiting the
"ownership" to just the two of us.
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