Skip to content

Rename dpcpp-nativecpu-pi-reviewers now that PI has been removed #15956

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

Closed
aarongreig opened this issue Nov 1, 2024 · 6 comments · Fixed by #16844
Closed

Rename dpcpp-nativecpu-pi-reviewers now that PI has been removed #15956

aarongreig opened this issue Nov 1, 2024 · 6 comments · Fixed by #16844
Assignees
Labels
enhancement New feature or request

Comments

@aarongreig
Copy link
Contributor

aarongreig commented Nov 1, 2024

PI has been removed, so the dpcpp-nativecpu-pi-reviewers team should be renamed to reflect that. Probably to something like dpcpp-nativecpu-ur-reviewers.

Once this has been done we'll also need to update the CODEOWNERS file

Copy link
Contributor

github-actions bot commented Jan 1, 2025

Hi! There have been no updates for at least the last 60 days, though the issue has assignee(s).

@stdale-intel, could you please take one of the following actions:

  • provide an update if you have any
  • unassign yourself if you're not looking / going to look into this issue
  • mark this issue with the 'confirmed' label if you have confirmed the problem/request and our team should work on it
  • close the issue if it has been resolved
  • take any other suitable action.

Thanks!

@KornevNikita
Copy link
Contributor

@stdale-intel @bader @PietroGhg @uwedolinsky hi! Could you please take a look?

@bader
Copy link
Contributor

bader commented Jan 29, 2025

Probably to something like dpcpp-nativecpu-ur-reviewers.

Considering that the same team owns code in LLVM and Clang projects. I suggest dropping ur from the name: dpcpp-nativecpu-reviewers. @aarongreig, are you okay with that?

@aarongreig
Copy link
Contributor Author

yeah I think that makes sense

@bader
Copy link
Contributor

bader commented Jan 30, 2025

I created internal tracker to make the change (DIO-19155).
@aarongreig, please, prepare a PR updating the team's name in the CODEOWNERS file.

@aarongreig
Copy link
Contributor Author

#16844

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants