Skip to content

Update code owners for frontend#8310

Closed
natke wants to merge 2 commits intomicrosoft:masterfrom
natke:update-codeowners
Closed

Update code owners for frontend#8310
natke wants to merge 2 commits intomicrosoft:masterfrom
natke:update-codeowners

Conversation

@natke
Copy link
Copy Markdown
Contributor

@natke natke commented Jul 6, 2021

Description: Update the codeowners files so appropriate people are assigned for review

@natke natke requested a review from a team as a code owner July 6, 2021 21:58
Comment thread CODEOWNERS
* @Microsoft/onnxruntime

# Python frontend owners
orttraining/*.py @thiagocrepaldi @spandantiwari @BowenBao @liqunfu @baijumeswani @mrry
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Might drop @mrry as well.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good point!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@nkreeger, would you want to be added as a code owner?

Copy link
Copy Markdown
Contributor

@thiagocrepaldi thiagocrepaldi left a comment

Choose a reason for hiding this comment

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

@natke do you know whether Github supports groups as opposed to individuals?

@natke
Copy link
Copy Markdown
Contributor Author

natke commented Jul 8, 2021

@natke do you know whether Github supports groups as opposed to individuals?

I don't know that. It would be easier to maintain, wouldn't it?

@SherlockNoMad
Copy link
Copy Markdown
Contributor

duplicate of #8335
will close this PR.

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