-
Notifications
You must be signed in to change notification settings - Fork 978
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
Adding CODEOWNERS.txt to identify lead maintainers. #1370
Conversation
CODEOWNERS.txt
Outdated
/oneTBB/src/tbb/ @pavelkumbrasev | ||
/oneTBB/src/tbb/ @dnmokhov | ||
/oneTBB/src/tbb/ @JhaShweta1 | ||
/oneTBB/src/tbb/ @sarathnandu | ||
/oneTBB/src/tbbbind/ @isaevil |
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.
What about /oneTBB/include/
, /oneTBB/integration
, /oneTBB/examples/
, /oneTBB/doc/
?
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.
@sarathnandu For doc/ please add @aepanchi
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.
What about
/oneTBB/include/
,/oneTBB/integration
,/oneTBB/examples/
,/oneTBB/doc/
?
The maintainers are specified component wise. Since these folders and files can be associated with a component, I didn't mention specific maintainers.
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.
@sarathnandu For doc/ please add @aepanchi
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.
Looks good for now. We'll update later as needed.
Description
Adding CODEOWNERS.txt to identify lead maintainers.
Fixes # - issue number(s) if exists
Type of change
Choose one or multiple, leave empty if none of the other choices apply
Add a respective label(s) to PR if you have permissions
Tests
Documentation
Breaks backward compatibility
Notify the following users
List users with
@
to send notificationsOther information