-
-
Notifications
You must be signed in to change notification settings - Fork 181
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
chore: add CODEOWNERS file #5213
Conversation
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.
Added myself for audit logs. LGTM
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.
LGTM!
Looks good. I will wait for everyone to take what they want, and I will have the rest 🤑 |
Remove my name from files I never touched, added @bufke, @jnm and @JacquelineMorrissette for files they should own.
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.
@jnm, @bufke , @JacquelineMorrissette please review my changes
…and John, and group these entries together
…as the backend lead developer. `CONTRIBUTING.md` applies equally to the backend, and `kpi.code-workspace` contains various settings related to formatting and linting Python code
@noliveleger, I have contradicted a few of them. You can see my rationale in my commit messages. LMK if you want to discuss anything |
Sorted, grouped and clarified that last line wins. Added the missing github folder, set @p2edwards as owner for patches folder as per his 👍 above. |
Checklist
./python-format.sh
to make sure that your code lints and that you've followed our coding styleDescription
Internal process configuration with no changes to the product.
Notes
Let's declare ownership of code. Please comment or directly push to this PR to add yourself (or others) to CODEOWNERS before approving. Read more in internal docs. In short:
Testing
Not testable. After merging, should suggest reviewers based on the CODEOWNERS file according to github docs.