Skip to content
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

Keep CODEOWNERS file up to date. #8500

Merged
merged 5 commits into from
Jul 20, 2021
Merged

Keep CODEOWNERS file up to date. #8500

merged 5 commits into from
Jul 20, 2021

Conversation

tqchen
Copy link
Member

@tqchen tqchen commented Jul 19, 2021

The CODEOWNERS file was used as a mechanism to mark committers' area
of expertises and faciliate the review process. This PR attempts to
bring its state to up to date. This is of course non-comprehensive,
but can serve as a starting pt to help us to find the right person
to shepherd the PRs.

@tqchen
Copy link
Member Author

tqchen commented Jul 19, 2021

cc @jroesch @junrushao1994

The CODEOWNERS file was used as a mechanism to mark committers' area
of expertises and faciliate the review process. This PR attempts to
bring its state to up to date. This is of course non-comprehensive,
but can serve as a starting pt to help us to find the right person
to shepherd the PRs.
Copy link
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

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

Minor nit. Great idea to update this file @tqchen

.github/CODEOWNERS Outdated Show resolved Hide resolved
Co-authored-by: Leandro Nunes <leandro.nunes@arm.com>
@tqchen
Copy link
Member Author

tqchen commented Jul 19, 2021

cc @apache/tvm-committers

Copy link
Contributor

@areusch areusch left a comment

Choose a reason for hiding this comment

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

thanks @tqchen!

.github/CODEOWNERS Outdated Show resolved Hide resolved
.github/CODEOWNERS Outdated Show resolved Hide resolved
Copy link
Contributor

@comaniac comaniac left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks

.github/CODEOWNERS Outdated Show resolved Hide resolved
Co-authored-by: Cody Yu <comaniac0422@gmail.com>
.github/CODEOWNERS Outdated Show resolved Hide resolved
Co-authored-by: Chenfan <jcf94@outlook.com>
@tqchen
Copy link
Member Author

tqchen commented Jul 20, 2021

Going to merge this, but future changes are more than welcomed.

@tqchen tqchen merged commit 2b74f97 into apache:main Jul 20, 2021
ylc pushed a commit to ylc/tvm that referenced this pull request Sep 29, 2021
* Keep CODEOWNERS file up to date.

The CODEOWNERS file was used as a mechanism to mark committers' area
of expertises and faciliate the review process. This PR attempts to
bring its state to up to date. This is of course non-comprehensive,
but can serve as a starting pt to help us to find the right person
to shepherd the PRs.

* Update .github/CODEOWNERS

Co-authored-by: Leandro Nunes <leandro.nunes@arm.com>

* Update CODEOWNERS

* Update .github/CODEOWNERS

Co-authored-by: Cody Yu <comaniac0422@gmail.com>

* Update .github/CODEOWNERS

Co-authored-by: Chenfan <jcf94@outlook.com>

Co-authored-by: Leandro Nunes <leandro.nunes@arm.com>
Co-authored-by: Cody Yu <comaniac0422@gmail.com>
Co-authored-by: Chenfan <jcf94@outlook.com>
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 13, 2022
* Keep CODEOWNERS file up to date.

The CODEOWNERS file was used as a mechanism to mark committers' area
of expertises and faciliate the review process. This PR attempts to
bring its state to up to date. This is of course non-comprehensive,
but can serve as a starting pt to help us to find the right person
to shepherd the PRs.

* Update .github/CODEOWNERS

Co-authored-by: Leandro Nunes <leandro.nunes@arm.com>

* Update CODEOWNERS

* Update .github/CODEOWNERS

Co-authored-by: Cody Yu <comaniac0422@gmail.com>

* Update .github/CODEOWNERS

Co-authored-by: Chenfan <jcf94@outlook.com>

Co-authored-by: Leandro Nunes <leandro.nunes@arm.com>
Co-authored-by: Cody Yu <comaniac0422@gmail.com>
Co-authored-by: Chenfan <jcf94@outlook.com>
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.

6 participants