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

COMMUNITY-ROLES: document CODEOWNERS file #11646

Merged
merged 7 commits into from
Dec 15, 2023

Conversation

sbrl
Copy link
Member

@sbrl sbrl commented Dec 2, 2023

Ref https://matrix.to/#/!zXiOpjSkFTvtMpsenJ:gitter.im/$P2PB65z9pBYv7KXAcTVv8p4WK_v3r1mx5cMyw5sgF2o?via=gitter.im&via=matrix.org&via=one.ems.host, this PR documents the .github/CODEOWNERS file in COMMUNITY-ROLES.

It is of course completely everyones' choice as to whether they are in the CODEOWNERS file or not.

@github-actions github-actions bot added the documentation Issues/PRs modifying the documentation. label Dec 2, 2023
Copy link
Member

@kbdharun kbdharun 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 for your contribution.

COMMUNITY-ROLES.md Outdated Show resolved Hide resolved
COMMUNITY-ROLES.md Outdated Show resolved Hide resolved
COMMUNITY-ROLES.md Outdated Show resolved Hide resolved
COMMUNITY-ROLES.md Outdated Show resolved Hide resolved
@kbdharun kbdharun mentioned this pull request Dec 2, 2023
70 tasks
@kbdharun kbdharun changed the title COMMUNITY-ROLES: Document CODEOWNERS file COMMUNITY-ROLES: document CODEOWNERS file Dec 2, 2023
sbrl and others added 3 commits December 4, 2023 21:14
Co-authored-by: K.B.Dharun Krishna <kbdharunkrishna@gmail.com>
Co-authored-by: Sebastiaan Speck <12570668+sebastiaanspeck@users.noreply.github.com>
Co-authored-by: Sebastiaan Speck <12570668+sebastiaanspeck@users.noreply.github.com>
COMMUNITY-ROLES.md Outdated Show resolved Hide resolved
Copy link
Member

@kbdharun kbdharun left a comment

Choose a reason for hiding this comment

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

I think we can add a line or note saying something like (at maybe the start or the sections end):

Note

This mechanism is purely for automatic review requests for PRs and doesn't grant collaborators additional copyright over the code-owned files. View LICENSE file for more information.

This clause would avoid confusion with people who aren't aware of code owners and might think we dual license our code for contributors (granting them additional rights).

sbrl and others added 2 commits December 12, 2023 18:46
Co-authored-by: Sebastiaan Speck <12570668+sebastiaanspeck@users.noreply.github.com>
Copy link
Member

@kbdharun kbdharun left a comment

Choose a reason for hiding this comment

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

The changes LGTM, great work @sbrl. I think it is GTG after the above review suggestion is applied.

COMMUNITY-ROLES.md Outdated Show resolved Hide resolved
Co-authored-by: K.B.Dharun Krishna <kbdharunkrishna@gmail.com>
@kbdharun kbdharun merged commit 508cea7 into main Dec 15, 2023
8 checks passed
@kbdharun kbdharun deleted the docs-community-roles-codeowners branch December 15, 2023 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Issues/PRs modifying the documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants