-
Notifications
You must be signed in to change notification settings - Fork 74.1k
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
[NVIDIA TF:XLA:JAX] Add NVIDIA as a Trusted Partner and assign reviewers #56704
Conversation
assignees.push('cheshire', 'cforster', 'reedwm', 'csigg'); | ||
} | ||
if (title.toLowerCase().includes("tf")) { | ||
assignees.push('rohanj', 'bfontain', 'penporn'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please check the Googler ldaps? Some appear to have different usernames linked to their github account.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GitHub usernames: hawkinsp, cheshire, gcforster, reedwm, chsigg, rohan100jain, bfontain, penpornk
Can you please update accordingly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for confirming, done!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for sending this PR! Can you update the GitHub usernames?
assignees = apple_silicon_assignees; | ||
if (title && title.toLowerCase().includes("nvidia") && domain.includes("nvidia.com")) { | ||
if (title.toLowerCase().includes("jax")) { | ||
assignees.push('hawkinsp'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add yashk2810
for jax too :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure thing, done!
assignees = apple_silicon_assignees; | ||
if (title && title.toLowerCase().includes("nvidia") && domain.includes("nvidia.com")) { | ||
if (title.toLowerCase().includes("jax")) { | ||
assignees.push('hawkinsp', 'yashk2810'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can also help out here, please add 'skye'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, added! Thank you for offering!
…in check to prevent accidental cross-partner triggering of assignments.
Add NVIDIA as a Trusted Partner and assign reviewers. Also add a domain check to prevent accidental cross-partner triggering of assignments.
Per-component designated reviewers:
The assignment logic retains the possibility of dispatching changes to more than one component, e.g. [NVIDIA TF XLA].
cc: @hawkinsp @yashk2810 @cheshire @gcforster @reedwm @chsigg @rohan100jain @bfontain @penpornk @skye